senorblanco@chromium.org 64cc579efa The plain Makefile was using -Wall, but the gyp build wasn't. This CL turns on ... -Wall -Wextra and -Wno-unused in common.gypi. This revealed a lot of warnings (and some actual bugs), all of which I fixed here. This is pretty mindless stuff for the most part (order of intialization, missing initializers, && within ||, etc), but will allow us to build cleanly with -Wall and -Wextra (and -Werror, if we so choose). I put defaults into switches that were missing cases. I could put in the actual missing enums instead if that's desired. I could also assert on missing enums instead of break, if that's desired. I wasn't sure how to test the stuff in "animator", so that should be looked at a bit more closely. Review URL: http://codereview.appspot.com/4547055/ git-svn-id: http://skia.googlecode.com/svn/trunk@1386 2bbb7eff-a529-9590-31e7-b0007b416f81		2011-05-19 19:58:58 +00:00
..
common.gypi	The plain Makefile was using -Wall, but the gyp build wasn't. This CL turns on	2011-05-19 19:58:58 +00:00
gyp_skia	This patch provides the include file common.gypi (cargo culted from another	2011-04-11 15:59:47 +00:00
skia.gyp	Add build for unix to gyp.	2011-05-19 19:02:20 +00:00