Bug 1693997 Comment 3 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
LGTM

Line is a bit long
> \+    var numAddresses = MailServices.headerParser.parseHeadersWithArray(aAddressesToAdd, emailAddresses, names, fullNames);

Maybe
> \+    var numAddresses =
> \+      MailServices.headerParser.parseHeadersWithArray(aAddressesToAdd,
> \+                                                      emailAddresses, names,
> \+                                                      fullNames);

But r/a+ either way.
Revision 1 by
on 
LGTM

Line is a bit long
> \+    var numAddresses = MailServices.headerParser.parseHeadersWithArray(aAddressesToAdd, emailAddresses, names, fullNames);

Maybe
\ > \+    var numAddresses =
\ > \+      MailServices.headerParser.parseHeadersWithArray(aAddressesToAdd,
\ > \+                                                      emailAddresses, name,
\ > \+                                                      fullNames);

But r/a+ either way.
Revision 2 by
on 
LGTM

Line is a bit long
> \+    var numAddresses = MailServices.headerParser.parseHeadersWithArray(aAddressesToAdd, emailAddresses, names, fullNames);

Maybe
> \+    var numAddresses =
> \+      MailServices.headerParser.parseHeadersWithArray(aAddressesToAdd,
> \+                                                      emailAddresses, name,
> \+                                                      fullNames);

But r/a+ either way.

Back to Bug 1693997 Comment 3