[webkit-dev] Features defines: Platform.h vs build system
Balazs Kelemen
kbalazs at webkit.org
Mon Oct 3 06:35:54 PDT 2011
Hi all!
I think we lack a consistent way for setting up the set of feature
defines. Some of them are handled via Platform.h and some of them via
the build systems. Don't you think all should be handled the same way?
This has came up in my mind in connection with the newly introduced
CSS_FILTERS define. (https://bugs.webkit.org/show_bug.cgi?id=68652) The
logic to set the define has only been added to the Mac build system
because the feature is Mac-only currently. I don't think it is a good
solution. As more platforms would use the feature they also need to add
the logic to their build system that is somewhat more difficult than
just adding one more ifdef branch to Platform.h. Note that it is not
totally avoidable to have some overhead per platform because we need to
handle the --css-filters build-webkit switch. However build-webkit
already knows how to set up a -D flag on each platform so it's not a big
deal.
I wonder about your opinion on this.
-kbalazs
More information about the webkit-dev
mailing list