[GTK] Multiple feature defines in webcore_cppflags can occur
Created attachment 162442 [details] Patch
Hi Martin, I'd be thankful for a review of this workaround.
Comment on attachment 162442 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=162442&action=review Looks good, though I have a small suggestion for a rename. > Source/WebCore/GNUmakefile.am:773 > +feature_defines_unstable_features += How about unstable_feature_defines? It avoids using the word "features" twice in this variable name.
Comment on attachment 162442 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=162442&action=review >> Source/WebCore/GNUmakefile.am:773 >> +feature_defines_unstable_features += > > How about unstable_feature_defines? It avoids using the word "features" twice in this variable name. Yes, the repetition should be avoided. How about feature_defines_unstable, though? Just so to keep the 'feature_defines_' prefix.
(In reply to comment #4) > (From update of attachment 162442 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=162442&action=review > Yes, the repetition should be avoided. How about feature_defines_unstable, though? Just so to keep the 'feature_defines_' prefix. I think we should try to move away from unnatural English in variable names where possible, but this is fine with me.
Committed r127872: <http://trac.webkit.org/changeset/127872>