You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Patrick Steinhardt <pa...@elegosoft.com> on 2016/10/12 13:05:15 UTC
[PATCH] Remove unused variables in conflict resolution code
Hi,
see the attached patch for fixes to some unused-variable
warnings.
[[
Remove variables which are written but not used in any way.
* subversion/libsvn_client/conflicts.c:
Remove unused variables
]]
--
Patrick Steinhardt, Entwickler
elego Software Solutions GmbH, http://www.elego.de
Geb�ude 12 (BIG), Gustav-Meyer-Allee 25, 13355 Berlin, Germany
Sitz der Gesellschaft: Berlin, USt-IdNr.: DE 163214194
Handelsregister: Amtsgericht Charlottenburg HRB 77719
Gesch�ftsf�hrer: Olaf Wagner
Re: [PATCH] Remove unused variables in conflict resolution code
Posted by Patrick Steinhardt <pa...@elegosoft.com>.
On Wed, Oct 12, 2016 at 03:46:38PM +0200, Ivan Zhakov wrote:
> On 12 October 2016 at 15:05, Patrick Steinhardt
> <pa...@elegosoft.com> wrote:
> > Hi,
> >
> > see the attached patch for fixes to some unused-variable
> > warnings.
> >
> > [[
> > Remove variables which are written but not used in any way.
> >
> > * subversion/libsvn_client/conflicts.c:
> > Remove unused variables
> > ]]
> Hi, Patrick.
>
> The log message should mention all affected functions/symbols. See
> 'Writing log messages' section in
> Community Guide [1]
>
> [1] https://subversion.apache.org/docs/community-guide/conventions.html#log-messages
>
> --
> Ivan Zhakov
Hi,
here's the updated commit message. Thanks for reviewing.
[[
Remove variables which are written but not used in any way.
* subversion/libsvn_client/conflicts.c:
(conflict_tree_get_incoming_description_generic,
conflict_tree_get_description_incoming_edit,
resolve_incoming_move_dir_merge,
configure_option_incoming_delete_accept,
configure_option_incoming_move_file_merge,
configure_option_incoming_dir_merge): Remove unused variables
]]
Regards
--
Patrick Steinhardt, Entwickler
elego Software Solutions GmbH, http://www.elego.de
Gebäude 12 (BIG), Gustav-Meyer-Allee 25, 13355 Berlin, Germany
Sitz der Gesellschaft: Berlin, USt-IdNr.: DE 163214194
Handelsregister: Amtsgericht Charlottenburg HRB 77719
Geschäftsführer: Olaf Wagner
Re: [PATCH] Remove unused variables in conflict resolution code
Posted by Ivan Zhakov <iv...@visualsvn.com>.
On 12 October 2016 at 15:05, Patrick Steinhardt
<pa...@elegosoft.com> wrote:
> Hi,
>
> see the attached patch for fixes to some unused-variable
> warnings.
>
> [[
> Remove variables which are written but not used in any way.
>
> * subversion/libsvn_client/conflicts.c:
> Remove unused variables
> ]]
Hi, Patrick.
The log message should mention all affected functions/symbols. See
'Writing log messages' section in
Community Guide [1]
[1] https://subversion.apache.org/docs/community-guide/conventions.html#log-messages
--
Ivan Zhakov