During the r1767197 thread, I noticed that vwrite_tuple() doesn't enforce its precondition that in the optional part of a tuple, either all values are valid or no value is; a call such as
vwrite_tuple("(?r?r)", SVN_INVALID_REVNUM, (svn_revnum_t) 42)
would happily generate a «( 42 ) » tuple, instead of triggering an
SVN_ERR_MALFUNCTION().
This would be easy to fix, and would prevent a class of bugs.
Cheers,
Daniel
