GNOME Bugzilla – Bug 697816
Coverity fixes
Last modified: 2013-05-02 21:25:58 UTC
Spotted by Coverity, etc.
Created attachment 241268 [details] [review] arg: Remove some dead code Spotted by Coverity.
Created attachment 241269 [details] [review] function: Fix some baaad integer signedness issues As these variables indicate -1 for "non-existant", they can't be unsigned.
Created attachment 241270 [details] [review] cairo-pattern: Remove some dead code Spotted by Coverity
Review of attachment 241268 [details] [review]: That's pretty clever that it detects that. Yes.
Review of attachment 241269 [details] [review]: Ouch. That is bad, but why int8 instead of int, which is what the functions actually return? Granted we can't really have functions with > 256 arguments, but still
Review of attachment 241270 [details] [review]: Sure.
Attachment 241268 [details] pushed as 3554344 - arg: Remove some dead code Attachment 241269 [details] pushed as db487c7 - function: Fix some baaad integer signedness issues Attachment 241270 [details] pushed as 16a1345 - cairo-pattern: Remove some dead code