Re: [INTERM PATCH] Issue 571: A+D should be R in update

Christopher Ness <chris@nesser.org> writes:

> * subversion/include/svn_wc.h
> (svn_wc_notify_action_t): New enumerated type
> svn_wc_notify_update_replace

It's not a new enumerated type, it's a new value in an existing type.

Such a change may well contravene Subversion's ABI compatability
guidelines; it's likely to break existing notification callbacks. I
see that we added locking values to svn_wc_notify_action_t in 1.2, but
that didn't break things in quite the same way. A 1.1 client using
1.2 libraries would continue to work so long as no locks were
involved, whereas your proposed change is likely to break 1.2 clients
using 1.3 libraries. You may need to rev the notification API.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org