Re: Safe to add a field to svn_error_t?

On Fri, Dec 24, 2010 at 12:10 AM, Blair Zajac <blair_at_orcaware.com> wrote:
> The addition of "tracing" to svn_error_t in 1.7 currently uses a special
> message string "traced call".  Instead of doing this, can we safely add an
> svn_boolean_t to svn_error_t the indicates if it is a traced error?
>
> While svn_error_t is public, I'm hoping most people are creating them with
> svn_error_create*(), so if that's the case, then code written and compiled
> against svn <= 1.6 would work fine when linked against a 1.7 library.
>
> Thoughts?

We don't explicitly say "only use approved methods to create this
object", but we are pretty implicit about it, with an entire
documentation group for "Error creation and destruction". I'm on the
fence on this, but would lean toward "go ahead and extend the struct"
if pressured.

What problem are you trying to solve?

-Hyrum
Received on 2010-12-24 15:08:17 CET