Re: API change - make svn_tristate_to_word/_from_word() private?

On Wed, Jun 15, 2011 at 10:14, Julian Foad <julian.foad_at_wandisco.com> wrote:
> New in 1.7 in svn_types.h:
>  svn_tristate_t
>  svn_tristate_to_word()  # yields "true"/"false"/NULL
>  svn_tristate_from_word()  # takes "true"/"yes"/"on"/.../
>...
> At least, and at first, we should remove the second and third usages
> listed above.  Those usages are more akin to "svn_cmdline" or
> "svn_config" utilities, or could well be private.
>
> Further, I consider moving/renaming these two functions to be a bit more
> private even for their main use in svn_log_changed_path2_t.

Concur. Those things always made me a little hinky.

Cheers,
-g
Received on 2011-06-15 18:52:38 CEST