Stefan Sperling <> (stsp)

Patch Suggested
r27802, r27593, r27495, r27316, r25707, r25578 r28702, r27301

r28702 | jerenkrantz | 2007-12-29 12:58:18 -0600 (Sat, 29 Dec 2007)

Diff: Make -p ignore 'public'/'private'/'protected' C++/Java reserved words.

Suggested by: Stefan Sperling

* subversion/libsvn_diff/diff_file.c
  (svn_diff__file_output_baton_t): Add a field for list of strings to skip.
  (output_unified_line): Skip if the curp matches any glob in our skip list.
  (svn_diff_file_output_unified3): Initialize our skip list with some reserved
  words from C++/Java.

r27802 | kfogel | 2007-11-13 17:01:52 -0600 (Tue, 13 Nov 2007)

* Remove legacy libtool helper before copying new one.

Patch by: Stefan Sperling <>

r27593 | hwright | 2007-11-05 09:10:13 -0600 (Mon, 05 Nov 2007)

In the command line client, don't reference a pointer to a local variable
outside of its declared scope.

Found by: Barry Scott <>
Patch by: Stefan Sperling <>

* subversion/svn/copy-cmd.c
  (svn_cl__copy): Allocate the peg revision for each copy source on the heap,
  instead of just using a pointer to the loop variable (which changes each time
  through the loop).

r27495 | cmpilato | 2007-10-30 14:54:26 -0500 (Tue, 30 Oct 2007)

Teach 'svnlook diff' about the --extensions parameter (just like the
'svn diff' subcommand already accepts).

* subversion/svnlook/main.c
  (options_table): Add --extensions (-x) support.
  (cmd_table): Teach the 'diff' subcommand to accept --extensions (-x).
  (svnlook_opt_state): Add 'extensions' member.
  (svnlook_ctxt_t): Add 'diff_options' member.
  (print_diff_tree): Pass requested diff extensions to the diff library
    (using svn_diff_file_diff_2 now).
  (get_ctxt_baton): Populate context baton's diff_options member.
  (main): Parse and handle new --extensions (-x) option.

* subversion/tests/cmdline/
  (diff_ignore_whitespace, diff_ignore_eolstyle): New tests.
  (test_list): Add references to new tests.

Patch by: Stefan Sperling <>
          (Tweaked by me.)
Reviewed by: dlr

r27316 | hwright | 2007-10-22 16:31:42 -0500 (Mon, 22 Oct 2007)

* subversion/bindings/swig/perl/native/

Fix grammatical errors in the documentation.
Also improve wording here and there to make the documentation
more clear and accessible.
Also, there is no 'destroy' function in the code, the author
probably meant the 'DESTROY' function.

Patch by: Stefan Sperling <>
(tweaked by me)

r27301 | kfogel | 2007-10-21 13:57:40 -0500 (Sun, 21 Oct 2007)

* www/project_issues.html
  (write-access): Make it easier to figure out how to log in.

Suggested by: Stefan Sperling <>

r25707 | vgeorgescu | 2007-07-10 05:49:07 -0500 (Tue, 10 Jul 2007)

Fix issue #2429: "svn export" doesn't export externals from working copies.

Patch by: Stefan Sperling <>

* subversion/libsvn_client/export.c
  (copy_versioned_files): Add an 'ignore_externals' argument, and copy
   externals to the destination if it is FALSE.
  (svn_client_export4): Update call to copy_versioned_files().

* subversion/tests/cmdline/
  (export_wc_with_externals): New test.
  (test_list): Add export_wc_with_externals().

r25578 | kameshj | 2007-06-29 08:25:58 -0500 (Fri, 29 Jun 2007)

Fix compilation failure if apr was compiled without thread support.

* subversion/libsvn_subr/atomic.c
* subversion/libsvn_ra_svn/cyrus_auth.c
  Check APR_HAS_THREADS with #if and not #ifdef.
  APR_HAS_THREADS is _always_ defined, but 0 if thread support is disabled.

Patch by: Stefan Sperling <>