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

Re: svn diff fix for bug 2044

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Tue, 19 Mar 2013 13:09:15 +0000 (GMT)

Hi Gabriela.

Gabriela Gibson wrote:

> I've made some changes to meet some feature requests regarding svn diff.
>
> Could you please take a look and let me know if I'm on the right track?

> [[[
> Change "svn diff" to allow removal of "-u" and use of
> arbitrary strings in place of current hard-coded "-L" switch. This
> partially resolves.
> http://subversion.tigris.org/issues/show_bug.cgi?id=2044

For the record, the summary line of issue #2044 is 'Fully customizable external diff invocations'.  (I like to mention the summary alongside the number as I am not good at memorizing issue numbers.)  I'm curious about your patch because I
am interested in issue #2044 and would like to see how this particular
change would fit in.

Please could you tell me more precisely what your patch does and why?  Of course I could read carefully through your patch to discover the 'what', but not the 'why'.

Thanks.

- Julian

>
> * subversion/svn/cl.h
>   (svn_cl__opt_state_t) Add two new fields.
>
> * subversion/svn/svn.c
>   (svn_cl__longopt_t) Add new field.
>   (svn_cl__options) Add new field and help information.
>   (svn_cl__cmd_table) Add two new parameters.
>   (sub_main) Initialize new field.
>   (sub_main) Add new case.
>   (sub_main) Add conditional call to svn_config_set.
>
> * subversion/svn/diff-cmd.c
>   (svn_cl__diff) Add braces and minor indentation issue?
>   (svn_cl__diff) Change call to svn_client_diff6 to svn_client_diff7.
>
> * subversion/include/svn_config.h
>   () Add new declarations.
>
> * subversion/include/svn_io.h
>   (svn_io_run_diff2_2) Add new function.
>
> * subversion/include/svn_client.h
>   (svn_client_blame) Add new function. svn_client_diff7.
>
> * subversion/libsvn_client/diff.c
>   (diff_cmd_baton) Add new field.
>   (diff_content_changed) Replace call to svn_io_run_diff2 with
>     svn_io_run_diff2_2.
>   (svn_client_diff7) Add new function.
>
> * subversion/libsvn_subr/io.c
>   (svn_io_run_diff2_2) Add new function.
>
> ]]]
>
Received on 2013-03-19 14:09:51 CET

This is an archived mail posted to the Subversion Dev mailing list.