RE: svn commit: r1366083 - /subversion/trunk/tools/dev/unix-build/Makefile.svn

2012-07-27 Thread Bert Huijben
> -Original Message- > From: s...@apache.org [mailto:s...@apache.org] > Sent: donderdag 26 juli 2012 18:57 > To: comm...@subversion.apache.org > Subject: svn commit: r1366083 - /subversion/trunk/tools/dev/unix- > build/Makefile.svn > > Author: stsp > Date: Thu Jul 26 16:56:31 2012 > New

Heads-up: new typedef 'svn_merge_rangelist_t'

2012-07-27 Thread Julian Foad
Heads up: I'm going to add   typedef apr_array_header_t svn_merge_rangelist_t; There's no major impact; it doesn't change anything functionally. The rest of the details: The immediate reason is to improve the debugging experience: I can then make GDB display 'rangelist' values in a simple human

Re: Heads-up: new typedef 'svn_merge_rangelist_t'

2012-07-27 Thread Julian Foad
Actually, it'll be called "svn_rangelist_t", since we've already got lots of public "svn_rangelist_...()" functions. - Julian - Original Message - > From: Julian Foad > To: Subversion Development > Cc: > Sent: Friday, 27 July 2012, 10:36 > Subject: Heads-up: new typedef 'svn_merge_

RE: Heads-up: new typedef 'svn_merge_rangelist_t'

2012-07-27 Thread Bert Huijben
Please check the swig typemaps after applying this change. Swig does care about the exact types in the source, but in this case it probably helps in generating better bindings. Bert Huijben (Cell phone) From: Julian Foad Sent: 27-7-2012 21:25 To: Subversion Development Subject: Re: Heads-up: new t

Re: Heads-up: new typedef 'svn_merge_rangelist_t'

2012-07-27 Thread Greg Stein
Seems fine, and there should be no affect on the API/ABI. And thanks for not including the '*' in there... that was a very poor choice for the mergeinfo :-( On Fri, Jul 27, 2012 at 3:24 PM, Julian Foad wrote: > Actually, it'll be called "svn_rangelist_t", since we've already got lots of > public