On 12/15/2010 09:17 PM, Arwin Arni wrote:
Hi,

I was looking through the subversion/libsvn_wc/update_editor.c and noticed that there was a redundant member (added_with_history) in the file baton structure. This is not being set anywhere which results in some unreachable code. I have removed the member and the corresponding unreachable code. This was (as I gather from svn blame) introduced in r867451 by glasser to implement copy-on-update. I understand that some of this was reverted by cmpilato. I guess this should have been removed too. I have attached a patch and a log message. Please review and respond.

Regards,
Arwin Arni
Thanks Arwin.

Committed it in r1049819.

With regards
Kamesh Jayachandran

Reply via email to