cppCheck: clarifyCondition

Remove some warnings about:
   Boolean result is used in bitwise operation
   Suspicious condition (assignment + comparison)

Also reduce scope of a variable.

Change-Id: I553e5a09ed1d6276be61efbbb2c28d165a69af39
Signed-off-by: jailletc36 <christophe.jaillet@wanadoo.fr>
Reviewed-on: https://gerrit.libreoffice.org/1031Reviewed-by: Tor Lillqvist <tml@iki.fi>
Tested-by: Tor Lillqvist <tml@iki.fi>

sax/source/expatwrap/saxwriter.cxx

@@ -1307,7 +1307,7 @@ void SAXWriter::startCDATA(void) throw(SAXException, RuntimeException)
 
 void SAXWriter::endCDATA(void) throw (RuntimeException)
 {
-    if( ! m_bDocStarted | ! m_bIsCDATA)
+    if( ! m_bDocStarted || ! m_bIsCDATA)
     {
         SAXException except;
         except.Message = OUString(  "endCDATA was called without startCDATA"  );

sc/source/ui/miscdlgs/acredlin.cxx

@@ -402,7 +402,7 @@ SvTreeListEntry* ScAcceptChgDlg::InsertChangeAction(
     pNewData->nActionNo=pScChangeAction->GetActionNumber();
     pNewData->bIsAcceptable=pScChangeAction->IsClickable();
     pNewData->bIsRejectable=pScChangeAction->IsRejectable();
-    pNewData->bDisabled=!pNewData->bIsAcceptable | bDisabled;
+    pNewData->bDisabled=!pNewData->bIsAcceptable || bDisabled;
     pNewData->aDateTime=aDateTime;
     pNewData->nRow  = aRef.aStart.Row();
     pNewData->nCol  = aRef.aStart.Col();
@@ -579,7 +579,7 @@ SvTreeListEntry* ScAcceptChgDlg::InsertFilteredAction(
         pNewData->nActionNo=pScChangeAction->GetActionNumber();
         pNewData->bIsAcceptable=pScChangeAction->IsClickable();
         pNewData->bIsRejectable=pScChangeAction->IsRejectable();
-        pNewData->bDisabled=!pNewData->bIsAcceptable | bDisabled;
+        pNewData->bDisabled=!pNewData->bIsAcceptable || bDisabled;
         pNewData->aDateTime=aDateTime;
         pNewData->nRow  = aRef.aStart.Row();
         pNewData->nCol  = aRef.aStart.Col();

sw/source/filter/ww8/ww8par5.cxx

@@ -3464,12 +3464,12 @@ eF_ResT SwWW8ImplReader::Read_F_Hyperlink( WW8FieldDesc* /*pF*/, String& rStr )
     String sURL, sTarget, sMark;
     bool bDataImport = false;
     //HYPERLINK "filename" [switches]
-    bool bOptions=false;
 
     rStr = comphelper::string::stripEnd(rStr, 1);
 
     if (!bDataImport)
     {
+        bool bOptions = false;
         long nRet;
         _ReadFieldParams aReadParam( rStr );
         while( -1 != ( nRet = aReadParam.SkipToNextToken() ))
@@ -3477,7 +3477,7 @@ eF_ResT SwWW8ImplReader::Read_F_Hyperlink( WW8FieldDesc* /*pF*/, String& rStr )
             switch( nRet )
             {
                 case -2:
-                    if (!sURL.Len() & !bOptions)
+                    if (!sURL.Len() && !bOptions)
                         ConvertFFileName(sURL, aReadParam.GetResult());
                     break;