Currently the coding style is mostly similar to XPConnect, which is horrible. Should use the normal style.
Here are some things that likely need to be fixed: - name arguments aFoo, and name members mFoo - indentation, and the mode line - |nsFoo* bar| vs |nsFoo *bar|. Which one should we use? - any remaining trailing whitespace? Any others?
(In reply to Andrew McCreight [:mccr8] from comment #1) > - |nsFoo* bar| vs |nsFoo *bar|. Which one should we use? Left!