diff options
author | Caolán McNamara <caolanm@redhat.com> | 2015-02-12 11:41:49 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2015-02-12 11:41:49 +0000 |
commit | 5e62dc99bfa59654166609fa57141e4303c734e3 (patch) | |
tree | 925b1bb81ac11c5742a587b4f92496b5512f0559 /sal/rtl | |
parent | 5b59724d081e429f13bec8b4aee4d6da01c657af (diff) |
coverity#1268298 mark up code
Change-Id: I6e48e2593f93efd71e84a076a99457c3daf7d9e0
Diffstat (limited to 'sal/rtl')
-rw-r--r-- | sal/rtl/ustring.cxx | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/sal/rtl/ustring.cxx b/sal/rtl/ustring.cxx index 90e5ba14126f..cc9981260a8d 100644 --- a/sal/rtl/ustring.cxx +++ b/sal/rtl/ustring.cxx @@ -569,11 +569,12 @@ void SAL_CALL rtl_uString_newFromCodePoints( ++n; } } - /* coverity[dead_error_condition] - Builds on the assumption that sal_Int32 - uses 32 bit two's complement representation with wrap around (the - necessary number of UTF-16 code units will be no larger than 2 * - SAL_MAX_INT32, represented as sal_Int32 -2): */ + /* Builds on the assumption that sal_Int32 uses 32 bit two's complement + representation with wrap around (the necessary number of UTF-16 code + units will be no larger than 2 * SAL_MAX_INT32, represented as + sal_Int32 -2): */ if (n < 0) { + // coverity[dead_error_begin] - assumes wrap around *newString = NULL; return; } |