Re: [VOTE] Merge svn-auth-x509 branch to trunk?
On 8/12/14 3:55 PM, Bert Huijben wrote:
> Since WC-NG we tried not to introduce new functions with flag arguments as in
> general functions like that are hard to maintain, while it is easy to rev
> functions to add another separate argument. (Another less preferred option is
> using a struct with separate args)
> I remember arguments from gstein but I have a hard time finding a mail reference.
> I think this would be the first new flag style argument in a public function
> since 1.2 or so.... If possible I would try using a different pattern here.
My goal was to avoid adding a bunch of separate arguments. The current
upper/lower and colons/no-colons isn't too bad. But I was thinking there might
be other formats that maybe didn't use hex or something like that we might want
to support in the future. But if we're trying to avoid flags I can just have
two booleans for the same functionality.
Received on 2014-08-13 01:00:32 CEST
This is an archived mail posted to the Subversion Dev