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

Re: svn commit: r1203687 - /subversion/trunk/subversion/svn/status.c

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Fri, 18 Nov 2011 10:36:04 -0500

On 11/18/2011 10:26 AM, Daniel Shahaf wrote:
> If we add a status->foo flag (like we added the status->changelist bit
> in 1.4), we would have to explicitly add it to this if() too --- which is
> too easy to forget.

Do you have a concrete suggestion for making this future-proof? If not,
we'll simply have to trust that anyone brave enough to add such a new flag
is also either smart enough to review all consumers of the structure or
mature enough to deal with it when someone eventually blames him or her for
not doing so. :-)

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2011-11-18 16:36:38 CET

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

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