Re: Suspicious conditional

On 29.06.2011 09:19, Dmitry wrote:
> Hey.
>
> Currently CPicture::SetActiveFrame() reads:
>
> UINT prevframe = frame;
> prevframe--;
> if (prevframe< 0)
> prevframe = 0;
>
> here prefframe is unsigned, so it can't be< 0. This defect was detected using PVS-Studio static analysis tool.

Fixed in r21635.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2780562
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].