Re: svn commit: r1507044 -

2013-07-26 Thread Ben Reser
On Thu, Jul 25, 2013 at 11:42 PM, Roderich Schupp roderich.sch...@gmail.com wrote: Thanks for catching this, but with Ben's r1507155 (Fix possible duplication of work when using svn_hash_sets() macro) there's no change required here, right? Right, this is a bigger problem than just your

svn commit: r1507217 - in /subversion/trunk/subversion/bindings/cxxhl: src/aprwrap/pool.hpp src/exception.cpp tests/test_aprwrap.cpp

2013-07-26 Thread brane
Author: brane Date: Fri Jul 26 08:42:08 2013 New Revision: 1507217 URL: http://svn.apache.org/r1507217 Log: Introduce a pool wrapper for the iteration pool idiom in the C++HL APR wrapper. * subversion/bindings/cxxhl/src/aprwrap/pool.hpp (IterationPool): New class; a pool wrapper that hides its

Re: svn commit: r1507044 -

2013-07-26 Thread Roderich Schupp
On Thu, Jul 25, 2013 at 10:39 PM, Bert Huijben b...@qqmail.nl wrote: This patch will allocate the string twice, because hash puts evaluates its argument twice. Thanks for catching this, but with Ben's r1507155 (Fix possible duplication of work when using svn_hash_sets() macro) there's no

RE: svn commit: r1507044 -

2013-07-26 Thread Bert Huijben
-Original Message- From: Ben Reser [mailto:b...@reser.org] Sent: vrijdag 26 juli 2013 04:53 To: Bert Huijben Cc: Subversion Development; rsch...@apache.org; commits@subversion.apache.org Subject: Re: svn commit: r1507044 - On Thu, Jul 25, 2013 at 5:26 PM, Ben Reser

svn commit: r1507265 - in /subversion/branches/fsfs-format7/subversion/libsvn_fs_x: transaction.c transaction.h tree.c tree.h

2013-07-26 Thread stefan2
Author: stefan2 Date: Fri Jul 26 11:35:32 2013 New Revision: 1507265 URL: http://svn.apache.org/r1507265 Log: On the fsfs-format7 branch: Replicate the FS commit API change from FSFS in FSX. * subversion/libsvn_fs_x/tree.h (svn_fs_x__commit_txn): add set_timestamp parameter *

svn commit: r1507266 - /subversion/branches/fsfs-format7/subversion/libsvn_fs_x/fs.c

2013-07-26 Thread stefan2
Author: stefan2 Date: Fri Jul 26 11:37:18 2013 New Revision: 1507266 URL: http://svn.apache.org/r1507266 Log: On the fsfs-format7 branch: Fix a deprecation warning in FSX. * subversion/libsvn_fs_x/fs.c (svn_fs_x__init): upgrade to using svn_ver_check_list2 Modified:

svn commit: r1507267 - /subversion/branches/fsfs-format7/subversion/libsvn_fs_x/low_level.c

2013-07-26 Thread stefan2
Author: stefan2 Date: Fri Jul 26 11:39:58 2013 New Revision: 1507267 URL: http://svn.apache.org/r1507267 Log: On the fsfs-format7 branch: Fix one of the svnadmin test cases. * subversion/libsvn_fs_x/low_level.c (svn_fs_x__unparse_representation): produce the exact same textual

svn commit: r1507294 - in /subversion/branches/1.8.x-r1507044: ./ subversion/bindings/swig/perl/libsvn_swig_perl/swigutil_pl.c

2013-07-26 Thread rschupp
Author: rschupp Date: Fri Jul 26 12:40:57 2013 New Revision: 1507294 URL: http://svn.apache.org/r1507294 Log: On the 1.8.x-r1507044 branch, backport r1507044 from trunk, with the following changes: * subversion/bindings/swig/perl/libsvn_swig_perl/swigutil_pl.c: Resolve whitespace conflicts.

svn commit: r1507297 - /subversion/branches/1.8.x/STATUS

2013-07-26 Thread rschupp
Author: rschupp Date: Fri Jul 26 12:46:21 2013 New Revision: 1507297 URL: http://svn.apache.org/r1507297 Log: Add backport branch for proposed r1507044. Modified: subversion/branches/1.8.x/STATUS Modified: subversion/branches/1.8.x/STATUS URL:

svn commit: r1507300 - /subversion/branches/1.7.x-r1507044/

2013-07-26 Thread rschupp
Author: rschupp Date: Fri Jul 26 12:49:59 2013 New Revision: 1507300 URL: http://svn.apache.org/r1507300 Log: Create r1507044 backport branch. Added: subversion/branches/1.7.x-r1507044/ (props changed) - copied from r1507298, subversion/branches/1.7.x/ Propchange:

svn commit: r1507301 - in /subversion/branches/1.7.x-r1507044: ./ subversion/bindings/swig/perl/libsvn_swig_perl/swigutil_pl.c

2013-07-26 Thread rschupp
Author: rschupp Date: Fri Jul 26 12:58:24 2013 New Revision: 1507301 URL: http://svn.apache.org/r1507301 Log: On the 1.7.x-r1507044 branch, backport r1507044 from trunk, with the following changes: * subversion/bindings/swig/perl/libsvn_swig_perl/swigutil_pl.c: Use ye olde apr_hash_set instead

svn commit: r1507303 - /subversion/branches/1.7.x/STATUS

2013-07-26 Thread rschupp
Author: rschupp Date: Fri Jul 26 13:01:47 2013 New Revision: 1507303 URL: http://svn.apache.org/r1507303 Log: * STATUS: Add backport branch for r1507044. Modified: subversion/branches/1.7.x/STATUS Modified: subversion/branches/1.7.x/STATUS URL:

svn commit: r1507347 - /subversion/branches/fsfs-improvements/subversion/libsvn_fs_fs/low_level.c

2013-07-26 Thread philip
Author: philip Date: Fri Jul 26 15:11:35 2013 New Revision: 1507347 URL: http://svn.apache.org/r1507347 Log: * subversion/libsvn_fs_fs/low_level.c (svn_fs_fs__parse_representation): Handle leading underscore when parsing the uniquifier, check the whole uniquifier is parsed. Modified:

svn propchange: r1507347 - svn:log

2013-07-26 Thread philip
Author: philip Revision: 1507347 Modified property: svn:log Modified: svn:log at Fri Jul 26 15:13:43 2013 -- --- svn:log (original) +++ svn:log Fri Jul 26 15:13:43 2013 @@ -1,3 +1,5 @@ +On the fsfs-improvements branch:

svn commit: r1507366 - /subversion/trunk/subversion/include/svn_hash.h

2013-07-26 Thread breser
Author: breser Date: Fri Jul 26 16:36:04 2013 New Revision: 1507366 URL: http://svn.apache.org/r1507366 Log: Fix possible duplication of work when using svn_hash_gets() macro. This is similar to the situation described about svn_hash_sets() in r1507155. However, we can't use the fix used there

svn commit: r1507367 - /subversion/trunk/subversion/include/svn_config.h

2013-07-26 Thread breser
Author: breser Date: Fri Jul 26 16:47:37 2013 New Revision: 1507367 URL: http://svn.apache.org/r1507367 Log: * subversion/include/svn_config.h: (svn_hash_sets): Change temp variable name to avoid shadow warnings. Suggested by: philip Modified:

Re: svn commit: r1507044 -

2013-07-26 Thread Ben Reser
On Fri, Jul 26, 2013 at 12:22 AM, Ben Reser b...@reser.org wrote: I haven't nominated it for backport yet because I wanted to figure out what to do about svn_hash_gets(). But I ended up realizing that there isn't a good solution there. Just realized there's no reason to backport this. 1.8.x

svn propchange: r1507366 - svn:log

2013-07-26 Thread breser
Author: breser Revision: 1507366 Modified property: svn:log Modified: svn:log at Fri Jul 26 16:57:23 2013 -- --- svn:log (original) +++ svn:log Fri Jul 26 16:57:23 2013 @@ -2,7 +2,7 @@ Fix possible duplication of work

svn commit: r1507382 - /subversion/trunk/subversion/libsvn_subr/config_auth.c

2013-07-26 Thread steveking
Author: steveking Date: Fri Jul 26 17:45:52 2013 New Revision: 1507382 URL: http://svn.apache.org/r1507382 Log: Allow passing NULL for the config dir param in svn_config_walk_auth_data, which means using the default configuration directory path. * subversion/libsvn_subr/config_auth.c

svn commit: r1507383 - /subversion/branches/1.8.x/STATUS

2013-07-26 Thread steveking
Author: steveking Date: Fri Jul 26 17:55:01 2013 New Revision: 1507383 URL: http://svn.apache.org/r1507383 Log: * STATUS: Cast vote and nominate r1507382 Modified: subversion/branches/1.8.x/STATUS Modified: subversion/branches/1.8.x/STATUS URL:

svn commit: r1507390 - /subversion/trunk/subversion/include/svn_config.h

2013-07-26 Thread breser
Author: breser Date: Fri Jul 26 18:36:41 2013 New Revision: 1507390 URL: http://svn.apache.org/r1507390 Log: Revert r1507367. I committed the wrong file with unintended changes. Modified: subversion/trunk/subversion/include/svn_config.h Modified:

svn propchange: r1507367 - svn:log

2013-07-26 Thread breser
Author: breser Revision: 1507367 Modified property: svn:log Modified: svn:log at Fri Jul 26 18:37:40 2013 -- --- svn:log (original) +++ svn:log Fri Jul 26 18:37:40 2013 @@ -2,3 +2,6 @@ (svn_hash_sets): Change temp

svn commit: r1507391 - /subversion/trunk/subversion/include/svn_hash.h

2013-07-26 Thread breser
Author: breser Date: Fri Jul 26 18:39:21 2013 New Revision: 1507391 URL: http://svn.apache.org/r1507391 Log: * subversion/include/svn_config.h: (svn_hash_sets): Change temp variable name to avoid shadow warnings. Suggested by: philip [This is the commit that was intended to be committed in

svn propchange: r1507391 - svn:log

2013-07-26 Thread breser
Author: breser Revision: 1507391 Modified property: svn:log Modified: svn:log at Fri Jul 26 18:40:27 2013 -- --- svn:log (original) +++ svn:log Fri Jul 26 18:40:27 2013 @@ -1,4 +1,4 @@ -* subversion/include/svn_config.h:

svn commit: r1507413 - in /subversion/trunk/subversion/bindings/cxxhl: include/svncxxhl/exception.hpp src/cxxhl-private.hpp src/exception.cpp src/private/ src/private/exception.hpp tests/test_exceptio

2013-07-26 Thread brane
Author: brane Date: Fri Jul 26 19:47:27 2013 New Revision: 1507413 URL: http://svn.apache.org/r1507413 Log: Refactor svn_error_t wrapping in C++HL so that we do not have to expose the type name in the public API, and simplify memory management. *

svn commit: r1507415 - in /subversion/trunk: build.conf subversion/bindings/cxxhl/src/cxxhl-private.hpp subversion/bindings/cxxhl/src/private/exception-private.hpp subversion/bindings/cxxhl/src/privat

2013-07-26 Thread brane
Author: brane Date: Fri Jul 26 20:00:31 2013 New Revision: 1507415 URL: http://svn.apache.org/r1507415 Log: Fix private header name clash in C++HL. * build.conf (private-includes): Add subversion/bindings/cxxhl/src/private/*.hpp. * subversion/bindings/cxxhl/src/private/exception-private.hpp:

svn commit: r1507544 - /subversion/branches/1.8.x/STATUS

2013-07-26 Thread danielsh
Author: danielsh Date: Sat Jul 27 01:14:37 2013 New Revision: 1507544 URL: http://svn.apache.org/r1507544 Log: * STATUS: Approve the r1506040 group. Edit the r1507382 entry. Modified: subversion/branches/1.8.x/STATUS Modified: subversion/branches/1.8.x/STATUS URL:

svn commit: r1507573 - in /subversion/trunk/subversion/bindings/cxxhl: src/cxxhl-private.hpp src/exception.cpp src/private.hpp tests/test_exception.cpp

2013-07-26 Thread brane
Author: brane Date: Sat Jul 27 04:29:26 2013 New Revision: 1507573 URL: http://svn.apache.org/r1507573 Log: * subversion/bindings/cxxhl/src/private.hpp: renamed from cxxhl-private.hpp. * subversion/bindings/cxxhl/src/exception.cpp, subversion/bindings/cxxhl/tests/test_exception.cpp: Update

svn commit: r1507574 - in /subversion/trunk/subversion/bindings/cxxhl: include/svncxxhl/_compat.hpp src/aprwrap/pool.hpp tests/test_aprwrap.cpp

2013-07-26 Thread brane
Author: brane Date: Sat Jul 27 05:00:47 2013 New Revision: 1507574 URL: http://svn.apache.org/r1507574 Log: Make the C++HL Pool class co. explicitly non-copyable. * subversion/bindings/cxxhl/include/svncxxhl/_compat.hpp (compat::noncopyable): Define mixin, or use Boost's version if available.

svn commit: r1507576 - in /subversion/trunk/subversion/bindings/cxxhl/src/aprwrap: array.hpp hash.hpp

2013-07-26 Thread brane
Author: brane Date: Sat Jul 27 05:09:34 2013 New Revision: 1507576 URL: http://svn.apache.org/r1507576 Log: * subversion/bindings/cxxhl/src/aprwrap/hash.hpp (Hash): Remove copy constructors. The hash wrapper is trivially copyable and the default assignment operators and copy constructors