Created attachment 663742 [details] [diff] [review] Patch This breaks the build on a recent Clang.
Hm, my editor is configured to trim trailing white space on save, so a bunch of that got into the patch. Is it OK to slip them in with this patch?
Comment on attachment 663742 [details] [diff] [review] Patch This patch is wrong, sorry for the bug spam. sizeof(enum) is implementation defined, and Clang emits the warning because it knows how many elements nsCSSPseudoElements::Type has. I don't think we're ever going to reach INT16_MAX pseudo elements, but if we want to keep the assertion, we'll need to add -Wno-tautological-out-of-range-comparison to CXXFLAGS in the near future.
You need to log in before you can comment on or make changes to this bug.