Created attachment 586990 [details] [diff] [review]
This was requested during the discussion in dev-platform.
I think this needs s/initialized/uninitialized/
At least, right now when I build with this and FAIL_ON_WARNINGS=1 added in content/smil, I get the following build error when I hit content/smil:
> error: -Werror=initialized: no option -Winitialized
Also, "man g++" confirms that there's no "-Winitialized", but there is "-Wuninitialized"
(confirmed locally that s/initialized/uninitialized/ fixes the error mentioned in Comment 2 and allows me to build)
Also, it looks like the existing patch (which I think is broken per comment 2 & comment 3) has already landed:
Oh, and Mounir pushed a bustage fix after that to do the s/initialized/uninitialized/ already. Cool. That bustage-fix was:
I seem to be hitting some weird "undefined symbol" issues on our Mac OS X tinderboxen, which (so far) always happen right after a -Wuninitialized warning is printed out. I suspect (but am not sure) that it might be due to GCC 4.2 mishandling "-Werror -Wno-error=uninitialized".
Anyway, I filed bug 716787 on that.
and the followup