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

Re: [PATCH] remove redundant code

From: Paul Burba <pburba_at_collab.net>
Date: 2007-04-06 15:49:18 CEST

> -----Original Message-----
> From: Kamesh Jayachandran
> Sent: Friday, April 06, 2007 3:30 AM
> To: Paul Burba; Malcolm Rowe
> Cc: SVN Dev
> Subject: RE: Re: [PATCH] remove redundant code
>
> Nice point Paul.
>
> Yes this patch breaks the existing behaviour.
>
> But won't it be good to move this snippet of code(with a
> better comment) from svn binary to libsvn_client(as suggested
> by Malcom) so that other client implementation would be free
> from having this code.

Hi Kamesh,

Sorry, I didn't mean to imply I disagreed with Malcolm. As long as the
problem with the default URL peg revision doesn't occur in your
subsequent patch I see no problem.

Paul B.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Apr 6 15:49:48 2007

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.