Closed
Bug 1175245
Opened 9 years ago
Closed 9 years ago
Use range-based "for" loops in nsRefreshDriver.cpp
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla41
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: dholbert, Assigned: dholbert)
Details
Attachments
(1 file)
7.13 KB,
patch
|
tnikkel
:
review+
|
Details | Diff | Splinter Review |
While poking at bug 1164539 (and subsequently bug 1175228), I noticed nsRefreshDriver has some for loops which could stand to be simplified using range-based syntax.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8623231 -
Flags: review?(tnikkel)
Assignee | ||
Comment 2•9 years ago
|
||
(This patch layers on top of my patch in bug 1175228; adding dependency to reflect that.) Note that I left a few loops unconverted -- in particular: - Loops over mObservers (which is an array of arrays); the conversion seemed like it might make those loops less readable. - A loop over nsCOMArray<imgIContainer> imagesToRefresh -- I think nsCOMArray might not support range-based for loops. (At least, I got compile errors when I tried to convert that loop.
Updated•9 years ago
|
Attachment #8623231 -
Flags: review?(tnikkel) → review+
Comment 4•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f858f1ba0ea5
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in
before you can comment on or make changes to this bug.
Description
•