On Jun 12, 2011, at 1:36 AM, "Brian Neal" <bgn...@gmail.com> wrote:

> Hello -
> 
> Attached is a patch for issue #3919 [1]. Please review when you have a
> chance. Thanks!
> 
> Possible commit message:
> 
> [[[
> Fix issue #3919. During a merge of a property, add a check against the
> incoming new property value and the working copy value. If they
> already match, then the merge trivially succeeds.
> 
> * subversion/libsvn_wc/props.c
>  (apply_single_generic_prop_change): Do nothing if the incoming new
> property value already matches the working value.
> 
> ]]]
> 
> 
> [1]: http://subversion.tigris.org/issues/show_bug.cgi?id=3919
> 
> 
> Best regards,
> BN
> <issue3919.txt>

I think conditions can be combined to form one "if" statement without "else if".

Thanks and Regards
Noorul

Reply via email to