Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(120)

Issue 6519778499887104: Issue 2010 - Remove some uses of "for each" with "for of" (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 12 months ago by tschuster
Modified:
4 years, 11 months ago
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

Issue 2010 - Remove some uses of "for each" with "for of"

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -24 lines) Patch
M chrome/content/errors.html View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/content/ui/composer.js View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/content/ui/filters.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/content/ui/filters-filterview.js View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/content/ui/filters-subscriptionview.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/content/ui/sendReport.js View 1 3 chunks +11 lines, -11 lines 0 comments Download
M chrome/content/ui/sidebar.js View 1 5 chunks +7 lines, -7 lines 0 comments Download

Messages

Total messages: 3
tschuster
4 years, 11 months ago (2015-02-26 23:50:35 UTC) #1
Wladimir Palant
LGTM
4 years, 11 months ago (2015-02-27 19:50:16 UTC) #2
tschuster
4 years, 11 months ago (2015-02-28 12:44:30 UTC) #3
On 2015/02/27 19:50:16, Wladimir Palant wrote:
> LGTM

Thanks Wladimir.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 87257f5