Fixed doParse functions

Fixed doParse functions

For some reason, "success" wasn't set in a bunch of template specializations, causing it to be invalid when used later on.
Committed: https://code.google.com/p/skia/source/detail?r=7969

[sugoi, 2013-02-26 20:56:52]

https://codereview.appspot.com/7370052/diff/1/src/utils/SkRTConf.cpp
File src/utils/SkRTConf.cpp (right):

https://codereview.appspot.com/7370052/diff/1/src/utils/SkRTConf.cpp#newcode148
src/utils/SkRTConf.cpp:148: template <typename T> T doParse(const char *, bool
*success ) {
This is just an unused parameter I removed.

[sugoi, 2013-03-04 16:46:24]

https://codereview.appspot.com/7370052/diff/1/src/utils/SkRTConf.cpp
File src/utils/SkRTConf.cpp (right):

https://codereview.appspot.com/7370052/diff/1/src/utils/SkRTConf.cpp#newcode250
src/utils/SkRTConf.cpp:250: if (success) {
This is the line where success is currently used before being assigned a value
when T is one of {int, unsigned int, float, double}.

[Humper, 2013-03-04 20:58:53]

lgtm, thanks

On 2013/03/04 16:46:24, sugoi wrote:
> https://codereview.appspot.com/7370052/diff/1/src/utils/SkRTConf.cpp
> File src/utils/SkRTConf.cpp (right):
> 
>
https://codereview.appspot.com/7370052/diff/1/src/utils/SkRTConf.cpp#newcode250
> src/utils/SkRTConf.cpp:250: if (success) {
> This is the line where success is currently used before being assigned a value
> when T is one of {int, unsigned int, float, double}.

[sugoi, 2013-03-04 21:03:44]

Committed patchset #1 manually as r7969 (presubmit successful).