cvs commit: src UPDATING src/include fts.h src/lib/libc/gen Makefile.inc Symbol.map fts-compat.c fts-compat.h fts.3 fts.c src/sys/sys param.h

Tom Rhodes trhodes at FreeBSD.org
Mon Jan 28 13:54:17 PST 2008


On Mon, 28 Jan 2008 09:40:09 -1000
Juli Mallett <jmallett at FreeBSD.org> wrote:

> * Yar Tikhiy <yar at comp.chem.msu.su> [ 2008-01-28 ]
> 	[ Re: cvs commit: src UPDATING src/include fts.h src/lib/libc/gen Makefile.inc Symbol.map fts-compat.c fts-compat.h fts.3 fts.c src/sys/sys param.h ]
> > But, hey, have you seen the diffs in the first place?  You seem to
> > assume that my commit was only to kill the innocent int64_t.
> 
> Yes, actually.  I found some of the decisions strange, the work overall
> worthwhile and the "long long" change particularly gauche.  Several others have
> said things that I thought at the time, and I was following the discussion
> closely.  (Your bogus standards reasoning was, in particular, a red flag, and I
> suspect that that caught some people off-guard since your reasoning is
> fallacious and that you would have gotten less grief for the change if you had
> given less strange justifications (even no justifications) initially.)  You said
> that you had received no back-out request, so I made one.  Instead of
> acknowledging that you had now received a single backout request which you would
> not honor (and I was only referring to the "long long" bit, for clarity), you
> chose to grouch that I was rehashing old points (I do not believe backout
> requests should come without reasoning attached.)

I'm not sure I completely agree with a backout but I agree
with changing the types in a follow up commit.

-- 
Tom Rhodes


More information about the cvs-src mailing list