On Sun, Feb 2, 2014 at 5:01 PM, Anthony Jones <ajo...@mozilla.com> wrote:
> On 31/01/14 13:25, Nicholas Nethercote wrote:
>> indentation, obviously we should fix any other style problems at the
>> same time.)
>
> In order to make this happen I've run clang-format on XPConnect and
> uploaded it to bug 966840.

I was curious about how this might eventually affect the stuff I work
on, so I looked at a couple of files' changes in the patch. I found
several omissions of advice from the style guide; the style guide
should be expanded to specify what the recommended style is in more
cases. See my comments in the bug.

Cheers,
Brian
-- 
Mozilla Networking/Crypto/Security (Necko/NSS/PSM)
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to