W/ dependent bugs fixed, gfx/2d is warning-free on my machine. Filing this bug on marking it as FAIL_ON_WARNINGS.
Created attachment 711544 [details] [diff] [review] fix v1 Here's the patch. Not requesting review until I've got Try feedback, since there are probably a few other warnings hiding on our other platforms/compilers.
Created attachment 711731 [details] [diff] [review] fix v2: FAIL_ON_WARNINGS but not on MSVC First try run (w/ fix v1): https://tbpl.mozilla.org/?tree=Try&rev=1115ce9503fd On MSVC debug, that encountered some C4244 warnings for converting between floating-point types. There are probably more of these, and they're often annoying/hacky to fix, so I'm just making this non-MSVC for now. The try run also encountered some mac-specific (and clang-specific) warnings, which I filed new bugs for. Here's a try run with patches (or a partial patch, in one case) for all of those: https://tbpl.mozilla.org/?tree=Try&rev=0639787d493f All green, woot!
I verified that this compiles successfully on my local machine with GCC 4.7, now that bug 839383 has been resolved. However, my GCC 4.8 prerelease does produce one additional warning in gfx/2d (which this bug's patch treats as an error) -- I'd like to fix that warning before landing, so that this doesn't bust GCC 4.8 builds. I filed bug 857740 on fixing that warning, with a patch.
(In reply to Daniel Holbert [:dholbert] from comment #2) > On MSVC debug, that encountered some C4244 warnings for converting between > floating-point types. There are probably more of these, and they're often > annoying/hacky to fix, so I'm just making this non-MSVC for now. Bug 857863 is actually disabling C4244, which removes the need for the MSVC exemption here. So, once the tree's reopened, I'll land this (after Bug 857863) without the MSVC exemption.
Created attachment 733603 [details] [diff] [review] fix v3: FAIL_ON_WARNINGS (now without MSVC exemption) Darn, we've got some new mac-specific warnings in this directory. I filed bug 858274 and bug 858304 for those. Once those are addressed, here's the final ready-to-land patch for this bug (with the MSVC exemption removed, per comment 4). Try run (green aside from the mac issues): https://tbpl.mozilla.org/?tree=Try&rev=28a74a7ddfd4
Created attachment 814714 [details] [diff] [review] fix v4 (now in moz.build) Now that the (at one time) last warnings for this dir, in QuartzSupport.mm, are being fixed over in bug 924444, here's an updated version of the FAIL_ON_WARNINGS patch. (now adding the annotation to moz.build instead of Makefile.in, per bug 882859) Carrying forward r+