convert some 'for .. in Iterator(array)' constructs to for..of loops in /mail (not tests)

RESOLVED FIXED in Thunderbird 47.0

Status

Thunderbird
General
--
enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: aceman, Assigned: aceman)

Tracking

Trunk
Thunderbird 47.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

18.73 KB, patch
Magnus Melin
: review+
Details | Diff | Splinter Review
(Assignee)

Description

2 years ago
In some cases where we just need to loop over values of an array, enclosing it in Iterator() seems needless and can be converted to the newer for..of.
(Assignee)

Comment 1

2 years ago
Created attachment 8713989 [details] [diff] [review]
patch

Try run:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=5b32daf5e8c5
Attachment #8713989 - Flags: review?(mkmelin+mozilla)

Comment 2

2 years ago
Comment on attachment 8713989 [details] [diff] [review]
patch

Review of attachment 8713989 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, r=mkmelin
Attachment #8713989 - Flags: review?(mkmelin+mozilla) → review+
(Assignee)

Comment 3

2 years ago
https://hg.mozilla.org/comm-central/rev/e197472677d9
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 47.0
You need to log in before you can comment on or make changes to this bug.