[git commit master] math.h: fix trivial typo (missing !): !defined __NO_LONG_DOUBLE_MATH

Mike Frysinger vapier at gentoo.org
Sun Sep 6 06:27:54 UTC 2009


On Saturday 05 September 2009 18:12:08 Denys Vlasenko wrote:
> On Saturday 05 September 2009 23:57, Mike Frysinger wrote:
> > > > > -			fcntl(LogFile, F_SETFD, 1); /* 1 == FD_CLOEXEC */
> > > > > +			fcntl(LogFile, F_SETFD, FD_CLOEXEC);
> > > > >  			/* We don't want to block if e.g. syslogd is SIGSTOPed */
> > > > >  			fcntl(LogFile, F_SETFL, O_NONBLOCK | fcntl(LogFile, F_GETFL));
> > > > >  		}
> > > >
> > > > please try to avoid committing unrelated things
> > >
> > > I know.
> > > I am too lazy to do four commits for four one-liners :(
> >
> > this might have been merged with the CLOEXEC patch as a general cloexec
> > cleanup, but it certainly should not be stuck in with random other
> > changes regardless of the size.  i know it's more effort than was
> > required in the past, but the end result makes navigating git history a
> > hell of a lot easier (especially when looking for regressions).
>
> You are right.
>
> This particular fubar was accidental - I just forgot that I edited
> that line with FD_CLOEXEC.
>
> I didn't tell you that since it sounds like I'm looking for an excuse.
> I don't - I know it's wrong to mix changes like I do.

well i guess since we all agree, there isnt much point in me continuing to 
whine.  thanks for your work in the minuscule details :).
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.busybox.net/pipermail/uclibc/attachments/20090906/07546941/attachment.pgp>


More information about the uClibc mailing list