Coverity Fixlet - UNINIT_CTOR

Coverity Fixlet - UNINIT_CTOR

http://chromecoverity.mtv.corp.google.com:5467/cov.cgi?events=17510516&line=127&prec=%2Fcov.cgi%3Fc%3DAAAAAAHA5g%26checker%3D55_56%26class%3D1_2_5%26comp%3DThird-party%26file%3D%252Fskia%252F%26q%3D6%26runs%3D924%26t%3D6%26v%3D1&run=924&t=12&v=1#line122

CID=16834,14530,16769
BUG=
TEST=

[scr, 2011-06-21 19:17:01]

Please review this Coverity Fixlet in Skia.

I made an implementation file GrSamplerState.cpp and used initializer list to
init all members in all constructors.

[bsalomon, 2011-06-21 19:22:58]

On 2011/06/21 19:17:01, scr wrote:
> Please review this Coverity Fixlet in Skia.
> 
> I made an implementation file GrSamplerState.cpp and used initializer list to
> init all members in all constructors.

LGTM. Do you need me to commit it?

[reed1, 2011-06-21 19:33:14]

don't submit yet

Where all of the white-space changes? The indents have changed in most of the .h
file

Why the constructor changes? Does this fix a bug or make the resulting code
smaller?

Can you make WS changes (if necessary) in a separate CL from substantive
changes?

[scr, 2011-06-21 19:53:33]

Ok, how 'bout this minimal change - only the header - only init the variables,
which weren't previously inited.

[reed1, 2011-06-21 20:06:49]

Sweet.

LGTM

[bsalomon, 2011-06-21 20:32:26]

checked in at r1665