Re: [TSVN] Very minor patch suggestion for SVNStatus.cpp

Will Dean wrote:

> I just took an update today - you've done a great job on the /W4
> warnings! Shall we go to /WX now...?

/WX? When I look at the warning level settings in VS.NET I can't see a
/WX setting?

> I saw a change you made in SVNStatus (only noticed it because of a
> conflict with something of mine), and wondered if you might consider the
> appended patch as an alternative.
>
> This makes no functional difference to the code but is, I feel, slightly
> clearer about the intention and slightly easier to maintain.

That's really a more elegant solution to silent that warning. I didn't
even knew that such a macro exists :-O

Committed in revision 1913.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org