Summary: | Update KCanvasFiltersQuartz.mm to match style guidelines | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||||
Component: | SVG | Assignee: | Eric Seidel (no email) <eric> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | oliver | ||||||
Priority: | P4 | ||||||||
Version: | 420+ | ||||||||
Hardware: | Mac | ||||||||
OS: | OS X 10.4 | ||||||||
Attachments: |
|
Description
Eric Seidel (no email)
2005-12-11 00:50:50 PST
Created attachment 5025 [details]
Code cleanup to match style guidelines (mostly spacing changes)
Created attachment 5026 [details]
A version of the patch ignoring whitespace
Comment on attachment 5025 [details]
Code cleanup to match style guidelines (mostly spacing changes)
+static QString KCPreviousFilterOutputName = QString("__previousOutput__");
The above should just be:
static QString KCPreviousFilterOutputName = "__previousOutput__";
r=me on this change
Reading this code I see lots to comment on that has nothing to do with what the
patch changes -- maybe as part of code review.
Oliver might hate me... hopefully this won't cause to many conflicts w/ his patches in progress. |