Summary: | Remove more DeprecatedString usage | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||||
Component: | WebKit Misc. | Assignee: | Eric Seidel (no email) <eric> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | ||||||||
Priority: | P4 | ||||||||
Version: | 420+ | ||||||||
Hardware: | Mac | ||||||||
OS: | OS X 10.4 | ||||||||
Attachments: |
|
Description
Eric Seidel (no email)
2006-03-20 17:08:53 PST
its even. Created attachment 7201 [details]
Remove more DeprecatedString
Created attachment 7202 [details]
Same patch as above, ignoring whitespace
Comment on attachment 7201 [details]
Remove more DeprecatedString
- kdDebug( 6080 ) << "CSSParser::CSSParser this=" << this << endl;
+ kdDebug(6080) << "CSSParser::CSSParser this=" << this << endl;
- kdDebug( 6080 ) << "CSS parse error " << str << endl;
+ kdDebug(6080) << "CSS parse error " << str << endl;
We don't even have kdDebug in our tree, so this should be removed rather than reformatted.
There's a ton in this patch that has nothing to do with DeprecatedString.
- valid_primitive = ( validUnit( value, FLength, strict ) );
+ valid_primitive = (validUnit(value, FLength, strict));
I suggest removing the parentheses altogether here.
+ // qDebug(" new value: id=%d", id);
-// kdDebug( 6080 ) << " got style, variant and weight current=" << valueList->currentValue << endl;
+// kdDebug(6080) << " got style, variant and weight current=" << valueList->currentValue << endl;
- // kdDebug( 6080 ) << " got size" << endl;
+ // kdDebug(6080) << " got size" << endl;
+ //qDebug("value = %s, converted=%.2f", DeprecatedString((QChar *)t, length).latin1(), yylval->val);
Lets remove commented-out debugging code too.
|