diff options
| author | jailletc36 <christophe.jaillet@wanadoo.fr> | 2012-11-11 10:42:37 +0100 |
|---|---|---|
| committer | Tor Lillqvist <tml@iki.fi> | 2012-11-12 10:58:57 +0000 |
| commit | 273a60af9aae1a0b87d2ff43bdfb0cd849b29d27 (patch) | |
| tree | 874ebb1484c2ac1e4f20f1bf2bd7f0942f2842be /sax | |
| parent | fb9522a229c2d7b45d5d9d6e2c4e27ddfde351cf (diff) | |
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>
Diffstat (limited to 'sax')
| -rw-r--r-- | sax/source/expatwrap/saxwriter.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
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" ); |