Update KCanvasFiltersQuartz.mm to match style guidelines This is a simple patch (99% spacing changes) to make the kcanvas filter code closer match our style guidelines. I also made one small fix of a leak of CIContexts under error conditions (we were exiting early from a method which later performed cleanup).
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.