[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [VOTE] Merge svn-auth-x509 branch to trunk?

From: Ben Reser <ben_at_reser.org>
Date: Tue, 12 Aug 2014 15:59:57 -0700

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 mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.