Comment on attachment 8381157
protocol.patch

Looks good, thanks.  Can you adjust the spacing a bit to match the
surrounding code, as noted below, please?

>+
>+

Only one blank line, please.

>+  if ( mClass ==  eProtocolInfo ) {

No space after '(' or before ')'.
Only one space after "==".

>+    *_retval = nsGNOMERegistry::HandlerExists (mSchemeOrType.get());

No space before '('.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1197559

Title:
  Regression: Firefox forgot how to open apt urls

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/1197559/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to