[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: SteveKing <steveking_at_gmx.ch>
Date: 2004-11-05 19:38:03 CET

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
Received on Fri Nov 5 19:39:15 2004

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.