Darren J Moffat wrote: [CC:'ing website-disc...@opensolaris.org that someone checks why bugs.opensolaris.org is stuck again] > Roland Mainz wrote: > > Darren J Moffat wrote: > >> Roland Mainz wrote: > >>> Roland Mainz wrote: > >>>> Are there any objections that I move <stdbool.h> to <sys/stdbool.h> (and > >>>> create a new <stdbool.h> which sources <sys/stdbool.h>) that (new) > >>>> kernel code can use the new C99 datatype |bool| ([1]) ? Note that I do > >>>> not intent do replace |boolean_t| in existing code, only allow that new > >>>> stuff (optionally) can use this C99 feature... > >>>> > >>>> [1]=Note that |bool| differs from the traditional kernel-land > >>>> |boolean_t| datatype that it is smaller (e.g. > >>>> |sizeof(boolean_t)==sizeof(int)| vs. |sizeof(bool)==1|) and optionally > >>>> allows the compiler to do better optimisations on this datatype (since > >>>> it is allowed to assume that it only contains |true| and |false| as > >>>> values instead of the full range from |INT_MIN|-|INT_MAX| for > >>>> |boolean_t| (since this enum is treated as |int| by value-range > >>>> optimisations)) and usually reduces the size of executables (e.g. > >>>> partially cleaning-up libast's boolean usage with |bool| reduces the > >>>> size of libast.so.1 from 1177272 bytes to 1094488 bytes). > >>> I've uploaded a webrev for the change as > >>> http://cr.opensolaris.org/~gisburn/sys_stdbool_h/ ... > >>> ... can anyone review this patch, please ? > >> I have no issues with the idea or the implementation. You may list me > >> as a codereviewer on your RTI. > > > > Thanks! :-) > > > >> I'm assuming that the intent is that this is delivered as a single > >> changeset and not as part of some larger RTI. > > > > No, I try to get this commited as standalone change (e.g. no > > monster-all-in-one-giant-glob putback like the ksh93-integration update2 > > thing) - the (likely) first test consumer will be the exec module > > cleanup work (e.g. minor code cleanup, add mapfiles etc.). > > > > BTW: Can anyone try to find the CR# I filed for this change, please ? I > > need the bugster number to request a sponsor... > > I couldn't fine one. I got 25 hits for stdbool.h in the text of a bug > and none of them discuss this issue.
Sounds like submissions via http://bugs.opensolaris.org/ are "stuck" again... ... can anyone from the website team please take a look ? ---- Bye, Roland -- __ . . __ (o.\ \/ /.o) roland.ma...@nrubsig.org \__\/\/__/ MPEG specialist, C&&JAVA&&Sun&&Unix programmer /O /==\ O\ TEL +49 641 3992797 (;O/ \/ \O;) _______________________________________________ opensolaris-code mailing list opensolaris-code@opensolaris.org http://mail.opensolaris.org/mailman/listinfo/opensolaris-code
