1702934 - Port |Bug 462681 - mailWindowOverlay.js style/whitespace/indention/comment/linewrap cleanup. And some function simplifications| to SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, task)

Description

[Ian Neal]

Port the relevant parts of the following bugs to SeaMonkey:

• Bug 462681 - mailWindowOverlay.js style/whitespace/indention/comment/linewrap cleanup. And some function simplifications
• Bug 492501 - Unable to get next 500 news messages in newsgroups

Comment 1

[Ian Neal]

Attachment: 1702934-port462681-2538.patch

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.8
Risk to taking this patch (and alternatives if risky): none
String changes made by this patch: none

Comment 2

[Ian Neal]

Attachment: 1702934-port462681-2539.patch

Unbitrotted patch

Comment 3

[Frank-Rainer Grahl (:frg)]

Comment on attachment 9232996 [details] [diff] [review]
1702934-port462681-2539.patch

LGTM

Comment 4

[Frank-Rainer Grahl (:frg)]

Attachment: 1702934-2-2539.patch

As discussed. The file is now halfway formatted after pthe first part but full of inconsistencies. Ran it thru the linter and adjusted one case manually where indention in one function was still 4 blanks.

[Approval Request Comment]
Regression caused by (bug #): nothing
User impact if declined: developer becomes unhappy when seeing the source without it.
Testing completed (on m-c, etc.): 2.53.9b1 pre
Risk to taking this patch (and alternatives if risky): Formatting only.
String changes made by this patch: --

Comment 5

[Ian Neal]

Comment on attachment 9234157 [details] [diff] [review]
1702934-2-2539.patch

[Triage Comment]
Neat r/a=me

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/b24f9863858e
Port |Bug 462681 - mailWindowOverlay.js style/whitespace/indention/comment/linewrap cleanup. And some function simplifications| to SeaMonkey. r=frg
https://hg.mozilla.org/comm-central/rev/ae9a0e80b346
Lint mailWindowOverlay.js. r=IanN

Comment 7

[Frank-Rainer Grahl (:frg)]

Attachment: 1702934-2-v1_1-2539.patch

Patch 2 v1.1 (as rebased and checked in) with 3 more misindented functions (4 spaces) manually corrected. Carrying forward r/a+

Comment 8

[Frank-Rainer Grahl (:frg)]

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/4c922c7d044eeafb77cf8de6e2045dfc3a763098
Port |Bug 462681 - mailWindowOverlay.js style/whitespace/indention/comment/linewrap cleanup. And some function simplifications| to SeaMonkey. r=frg a=frg
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/fbb1b073abac97b7196ba1754250432c1df23fca
Lint mailWindowOverlay.js. r=IanN a=IanN