From 273a60af9aae1a0b87d2ff43bdfb0cd849b29d27 Mon Sep 17 00:00:00 2001
From: jailletc36 <christophe.jaillet@wanadoo.fr>
Date: Sun, 11 Nov 2012 10:42:37 +0100
Subject: 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/1031
Reviewed-by: Tor Lillqvist <tml@iki.fi>
Tested-by: Tor Lillqvist <tml@iki.fi>
---
 sax/source/expatwrap/saxwriter.cxx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'sax')

diff --git a/sax/source/expatwrap/saxwriter.cxx b/sax/source/expatwrap/saxwriter.cxx
index e54c9fcc1d22..1caf6cddbc3b 100644
--- a/sax/source/expatwrap/saxwriter.cxx
+++ b/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"  );
-- 
cgit