Am 19.09.2010 21:53, schrieb Quentin Mathé: > Le 19 sept. 2010 à 03:13, Gregory Casamento a écrit : > >> What does getting the CF stuff by default really hurt? If someone >> doesn't want it then just don't use it. Making it optional just >> complicates things a little. >> >> On the other hand, I guess it just adds more weight to the library if >> you know you're not going to use it in the first place. > > For now, I just added the standard C headers of CoreFoundation.h to > preface.h.in. > > This makes the compilation a bit slower: > > Base with new preface.h > real 0m48.585s > user 0m42.819s > sys 0m5.292s > > Base with old preface.h > real 0m45.288s > user 0m39.858s > sys 0m4.944s > > Gui with new preface.h > real 0m58.710s > user 0m51.835s > sys 0m6.368s > > Gui with old preface.h > real 0m54.698s > user 0m48.687s > sys 0m5.776s > > To keep the compilation time as it is, we can pass > -DCF_EXCLUDE_CSTD_HEADERS in Base, Gui and Back and continue to include > the C headers as we currently do it. We just need to include string.h in > few more places if we do that.
Sounds like a great idea to me. Up to now we made sure that GNUstep only includes the headers we really need and we should keep it that way for the internal code at least. > For Gui, HUGE macro is 3.40282347e+38F in math.h and 1e7 in > NSTextView.m. Since math.h is automatically included we now get a > warning. Can we use the value from math.h or not? There is a comment > stating the value should be changed carefully. We could easily rename the variable in NSTextView. As far as I remember the actual value isn't that important, it just has to be "huge", compared to a normal dimension of a text container. I just noticed that in many places we are using 1e7 directly. We should replace all these occurrences with a new variable and may then be free to change the actual value. _______________________________________________ Gnustep-dev mailing list Gnustep-dev@gnu.org http://lists.gnu.org/mailman/listinfo/gnustep-dev