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

Re: svn commit: r1634906 - in /subversion/branches/svn-auth-x509/subversion: include/svn_x509.h libsvn_subr/x509.h libsvn_subr/x509info.c libsvn_subr/x509parse.c svn/auth-cmd.c tests/libsvn_subr/x509-test.c

From: Ben Reser <ben_at_reser.org>
Date: Wed, 29 Oct 2014 18:00:53 -0700

On 10/28/14 9:26 AM, Branko Čibej wrote:
> Indentation is wrong.

Whoops, somehow my code style setting got disabled when I wrote this and didn't
notice it. Fixed in r1635359.

> And I suggest using svn_iter_apr_array here.
> ... and svn_iter_apr_hash here.

Unless it's a big deal to you I'm not inclined to use those here. They feel
very clunky for this use. For one thing I need to add another function for the
callback that have only one line. For another I have to pass the pool into the
callback baton since they both pass only a scratch pool.
Received on 2014-10-30 02:01:48 CET

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.