teach unnecessaryparen loplugin about identifiers

Change-Id: I5710b51e53779c222cec0bf08cd34bda330fec4b
Reviewed-on: https://gerrit.libreoffice.org/39737
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>

3 files changed, 15 insertions, 15 deletions

diff --git a/cui/source/dialogs/colorpicker.cxx b/cui/source/dialogs/colorpicker.cxx
index 548594aa1f9b..44c3d21c742b 100644
--- a/cui/source/dialogs/colorpicker.cxx
+++ b/cui/source/dialogs/colorpicker.cxx
@@ -1177,7 +1177,7 @@ IMPL_LINK_NOARG(ColorPickerDialog, ColorFieldControlModifydl, ColorFieldControl&
         break;
     }
 
-    update_color(UpdateFlags::All & ~(UpdateFlags::ColorChooser));
+    update_color(UpdateFlags::All & ~UpdateFlags::ColorChooser);
 }
 
 IMPL_LINK_NOARG(ColorPickerDialog, ColorSliderControlModifyHdl, ColorSliderControl&, void)
@@ -1205,7 +1205,7 @@ IMPL_LINK_NOARG(ColorPickerDialog, ColorSliderControlModifyHdl, ColorSliderContr
         break;
     }
 
-    update_color(UpdateFlags::All & ~(UpdateFlags::ColorSlider));
+    update_color(UpdateFlags::All & ~UpdateFlags::ColorSlider);
 }
 
 IMPL_LINK(ColorPickerDialog, ColorModifyEditHdl, Edit&, rEdit, void)
@@ -1215,52 +1215,52 @@ IMPL_LINK(ColorPickerDialog, ColorModifyEditHdl, Edit&, rEdit, void)
     if (&rEdit == mpMFRed)
     {
         setColorComponent( ColorComponent::Red, ((double)mpMFRed->GetValue()) / 255.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::RGB);
+        n = UpdateFlags::All & ~UpdateFlags::RGB;
     }
     else if (&rEdit == mpMFGreen)
     {
         setColorComponent( ColorComponent::Green, ((double)mpMFGreen->GetValue()) / 255.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::RGB);
+        n = UpdateFlags::All & ~UpdateFlags::RGB;
     }
     else if (&rEdit == mpMFBlue)
     {
         setColorComponent( ColorComponent::Blue, ((double)mpMFBlue->GetValue()) / 255.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::RGB);
+        n = UpdateFlags::All & ~UpdateFlags::RGB;
     }
     else if (&rEdit == mpMFHue)
     {
         setColorComponent( ColorComponent::Hue, (double)mpMFHue->GetValue() );
-        n = UpdateFlags::All & ~(UpdateFlags::HSB);
+        n = UpdateFlags::All & ~UpdateFlags::HSB;
     }
     else if (&rEdit == mpMFSaturation)
     {
         setColorComponent( ColorComponent::Saturation, ((double)mpMFSaturation->GetValue()) / 100.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::HSB);
+        n = UpdateFlags::All & ~UpdateFlags::HSB;
     }
     else if (&rEdit == mpMFBrightness)
     {
         setColorComponent( ColorComponent::Brightness, ((double)mpMFBrightness->GetValue()) / 100.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::HSB);
+        n = UpdateFlags::All & ~UpdateFlags::HSB;
     }
     else if (&rEdit == mpMFCyan)
     {
         setColorComponent( ColorComponent::Cyan, ((double)mpMFCyan->GetValue()) / 100.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::CMYK);
+        n = UpdateFlags::All & ~UpdateFlags::CMYK;
     }
     else if (&rEdit == mpMFMagenta)
     {
         setColorComponent( ColorComponent::Magenta, ((double)mpMFMagenta->GetValue()) / 100.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::CMYK);
+        n = UpdateFlags::All & ~UpdateFlags::CMYK;
     }
     else if (&rEdit == mpMFYellow)
     {
         setColorComponent( ColorComponent::Yellow, ((double)mpMFYellow->GetValue()) / 100.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::CMYK);
+        n = UpdateFlags::All & ~UpdateFlags::CMYK;
     }
     else if (&rEdit == mpMFKey)
     {
         setColorComponent( ColorComponent::Key, ((double)mpMFKey->GetValue()) / 100.0 );
-        n = UpdateFlags::All & ~(UpdateFlags::CMYK);
+        n = UpdateFlags::All & ~UpdateFlags::CMYK;
     }
     else if (&rEdit == mpEDHex)
     {
@@ -1278,7 +1278,7 @@ IMPL_LINK(ColorPickerDialog, ColorModifyEditHdl, Edit&, rEdit, void)
 
                 RGBtoHSV( mdRed, mdGreen, mdBlue, mdHue, mdSat, mdBri );
                 RGBtoCMYK( mdRed, mdGreen, mdBlue, mdCyan, mdMagenta, mdYellow, mdKey );
-                n = UpdateFlags::All & ~(UpdateFlags::Hex);
+                n = UpdateFlags::All & ~UpdateFlags::Hex;
             }
         }
     }
diff --git a/cui/source/dialogs/hlinettp.cxx b/cui/source/dialogs/hlinettp.cxx
index eb86a6629873..fe3f1ddd1abb 100644
--- a/cui/source/dialogs/hlinettp.cxx
+++ b/cui/source/dialogs/hlinettp.cxx
@@ -257,7 +257,7 @@ void SvxHyperlinkInternetTp::SetScheme(const OUString& rScheme)
 {
     //if rScheme is empty or unknown the default behaviour is like it where HTTP
     bool bFTP = rScheme.startsWith(sFTPScheme);
-    bool bInternet = !(bFTP);
+    bool bInternet = !bFTP;
 
     //update protocol button selection:
     m_pRbtLinktypFTP->Check(bFTP);
diff --git a/cui/source/options/optlingu.cxx b/cui/source/options/optlingu.cxx
index 4a65279ff477..a67c458e43ec 100644
--- a/cui/source/options/optlingu.cxx
+++ b/cui/source/options/optlingu.cxx
@@ -386,7 +386,7 @@ void OptionsUserData::SetNumericValue( sal_uInt8 nNumVal )
     if (HasNumericValue()  &&  (GetNumericValue() != nNumVal))
     {
         nVal &= 0xffffff00;
-        nVal |= (nNumVal);
+        nVal |= nNumVal;
         nVal |= (sal_uLong)1 << 11; // mark as modified
     }
 }