Re: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-09-01 Thread Daniel Shahaf
Backport review: cmpil...@apache.org wrote on Mon, Aug 20, 2012 at 15:19:06 -: > Author: cmpilato > Date: Mon Aug 20 15:19:06 2012 > New Revision: 1375052 > > URL: http://svn.apache.org/viewvc?rev=1375052&view=rev > Log: > Minor code simplifications. No logical changes. > > * subversion/lib

Re: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-08-20 Thread C. Michael Pilato
On 08/20/2012 12:00 PM, Bert Huijben wrote: > I like that change too. Maybe we should backport it to 1.7.x in some way? Agreed! I'll propose it in a bit if no one beats me to it. -- C. Michael Pilato CollabNet <> www.collab.net <> Enterprise Cloud Development signature.asc Descript

RE: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-08-20 Thread Bert Huijben
> -Original Message- > From: C. Michael Pilato [mailto:cmpil...@collab.net] > Sent: maandag 20 augustus 2012 17:44 > To: Bert Huijben > Cc: cmpil...@apache.org; dev@subversion.apache.org > Subject: Re: svn commit: r1375052 - > /subversion/trunk/subversion/libsvn_subr

Re: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-08-20 Thread C. Michael Pilato
On 08/20/2012 11:22 AM, Bert Huijben wrote: > Your log message says no 'logical changes', while this seems to remove > the error for unknown password stores? Doh! Good catch, Bert. Well, as it turns out, I like that change, so I'll just fix the log message. -- C. Michael Pilato CollabNet <

RE: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-08-20 Thread Bert Huijben
> -Original Message- > From: cmpil...@apache.org [mailto:cmpil...@apache.org] > Sent: maandag 20 augustus 2012 17:19 > To: comm...@subversion.apache.org > Subject: svn commit: r1375052 - > /subversion/trunk/subversion/libsvn_subr/auth.c > > Author: cmpilato > Date: Mon Aug 20 15:19:06 20