diff options
author | Tor Lillqvist <tml@iki.fi> | 2013-03-27 20:06:46 +0200 |
---|---|---|
committer | Tor Lillqvist <tml@iki.fi> | 2013-03-28 00:17:36 +0200 |
commit | a1f2f8636dd3b4b75bb24833bb42734a5f7c6c37 (patch) | |
tree | b4a5cccb67bc3184ea580f8134b3008757a0dd91 /desktop | |
parent | 3fcf0008d382b3f8677b02adfb3b01914d1123dc (diff) |
WaE: comparison is always true due to limited range of data type
Change-Id: I74d9025b37966d0a0fa1e2c0cd5cac4778f8384e
Diffstat (limited to 'desktop')
-rw-r--r-- | desktop/source/deployment/dp_persmap.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/desktop/source/deployment/dp_persmap.cxx b/desktop/source/deployment/dp_persmap.cxx index 05cd002f6ab5..aab763165982 100644 --- a/desktop/source/deployment/dp_persmap.cxx +++ b/desktop/source/deployment/dp_persmap.cxx @@ -79,8 +79,8 @@ static OString encodeString( const OString& rStr) // short circuit for the simple non-encoded case while( --i >= 0) { - const sal_Char c = *(pChar++); - if( (0x00 <= c) && (c <= 0x0F)) + const unsigned char c = (unsigned char) *(pChar++); + if( c <= 0x0F ) break; if( c == '%') break; @@ -93,8 +93,8 @@ static OString encodeString( const OString& rStr) aEncStr.append( pChar - (nLen-i), nLen - i); while( --i >= 0) { - sal_Char c = *(pChar++); - if( (0x00 <= c) && (c <= 0x0F)) + unsigned char c = (unsigned char) *(pChar++); + if( c <= 0x0F ) { aEncStr.append( '%'); c += (c <= 0x09) ? '0' : 'A'-10; |