RESOLVED FIXED 17201
Use RefPtr in CSSParser::parseFont() to get rid of goto silliness 
https://bugs.webkit.org/show_bug.cgi?id=17201
Summary Use RefPtr in CSSParser::parseFont() to get rid of goto silliness
David Kilzer (:ddkilzer)
Reported 2008-02-06 14:28:33 PST
CSSParser::parseFont() uses goto statements to release the FontValue object on early exit. It should be using a RefPtr<FontValue> instead.
Attachments
First attempt (3.62 KB, patch)
2008-02-10 23:31 PST, Rob Buis 		ddkilzer: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Rob Buis
Comment 1 2008-02-10 23:31:24 PST
Created attachment 19059 [details] First attempt Gets rid of the goto usage in parseFont. Cheers, Rob.
David Kilzer (:ddkilzer)
Comment 2 2008-02-11 06:23:22 PST
Comment on attachment 19059 [details] First attempt Thanks Rob!! r=me Please remove tabs from the ChangeLog before landing!
Rob Buis
Comment 3 2008-02-11 13:06:48 PST
Landed in 30152.
Note You need to log in before you can comment on or make changes to this bug.