Re: [PATCH] Fix issue #2809

Hi Julian,

Julian Foad wrote:
>>From reading the patch and the rest of the log message, I see that this
> patch leave the regular output just how it was, and adds an "--xml"
> option for giving XML output when the user wants it, which is the
> solution suggested in the issue tracker. It would be good to say this in
> the log message summary paragraph so that the reader does not have to go
> and investigate.

Updated the log message.

> Is there a reason for this difference?

Actually, I missed it during my initial development of this patch. Thanks for
pointing it out.

> There's one warning when I build with this patch:
>
> subversion/svnlook/main.c:1548: warning: no previous prototype for
> 'print_xml_prop'

The updated patch must not give this warning.

I ve attached an updated patch with the above changes and log message changed.
Julian, Thanks for your review.

-- 
Senthil Kumaran S
http://www.stylesen.org/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org