[umbrello-devel] [umbrello] [Bug 340646] Fix issues reported by Coverity scan

Oliver Kellogg okellogg at users.sourceforge.net
Sun Nov 30 19:07:14 UTC 2014


https://bugs.kde.org/show_bug.cgi?id=340646

--- Comment #1 from Oliver Kellogg <okellogg at users.sourceforge.net> ---
Git commit 9324cf30e4017d0f37a95d47da96882a8d713199 by Oliver Kellogg.
Committed on 30/11/2014 at 19:06.
Pushed by okellogg into branch 'master'.

Coverity check on master at 375ea16 CIDs 83526, 83527, 83528, 83529, 83530 :
Logically dead code in file unittests/TEST_basictypes.cpp

- CID 83526: At function test_Corner_toString_fromString() for-loop,
  - use "<=" as comparison operator for staying in loop;
  - replace ++i by i *= 2 on iteration action. Reason: The values of
    the Corner enum are represented as 0x1, 0x2, 0x4, 0x8.

- CID 83527: At function test_RoleType_toString_fromString() for-loop,
  use "<=" as comparison operator for staying in loop.

- CID 83528: At function test_RoleType_toString_fromString() for-loop,
  use "<=" as comparison operator for staying in loop.

- CID 83529: At function test_Region_toString_fromString() for-loop,
  use "<=" as comparison operator for staying in loop.

- CID 83530: At function test_ProgrammingLanguage_toString_fromString()
  for-loop, use "<=" as comparison operator for staying in loop.

M  +5    -5    unittests/TEST_basictypes.cpp

http://commits.kde.org/umbrello/9324cf30e4017d0f37a95d47da96882a8d713199

-- 
You are receiving this mail because:
You are the assignee for the bug.


More information about the umbrello-devel mailing list