svn commit: r363361 - in head/editors/fte: . files

Alexey Dokuchaev danfe at FreeBSD.org
Fri Aug 1 06:44:31 UTC 2014


On Fri, Aug 01, 2014 at 08:40:37AM +0200, John Marino wrote:
> On 8/1/2014 08:32, Alexey Dokuchaev wrote:
> > Well, my original idea (bail out with an error) is how I saw we prevent
> > overwriting: by asserting that output file does not exist; you think it
> > is not robust enough?
> 
> No, because you don't cover the case where both files legitimately need
> patching.  Why bail out with error when I *want* both patches to exist?
> You are forcing me to rename one?  to what

I see your point; I was thinking that if user will get this error, she will
choose a different character; but now I'll follow what you suggest.

> I've been using "_" => "__" for a while now, it's not bad.  Yes, double
> underscore remains but a very low frequency.  The main issue is that it
> represents a different character ("_") than it does now ("\").

OK, I agree.

./danfe


More information about the svn-ports-all mailing list