Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(221)

Issue 2154933002: Add example int doc for Iterable.expand (Closed)

Created:
4 years, 5 months ago by stanm
Modified:
4 years, 4 months ago
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/sdk@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Add example int doc for Iterable.expand The expand function is not trivial and an example helps flesh out its behaviour. BUG= R=floitsch@google.com Committed: https://github.com/dart-lang/sdk/commit/0648e2622b2cc7d960993e51024108600be8c0b0

Patch Set 1 #

Total comments: 2

Patch Set 2 : Address comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -5 lines) Patch
M sdk/lib/core/iterable.dart View 1 6 chunks +14 lines, -5 lines 0 comments Download

Messages

Total messages: 10 (3 generated)
stanm
CC'ing Lasse as he's the last person who's edited that comment.
4 years, 5 months ago (2016-07-15 22:56:09 UTC) #2
stanm
bump
4 years, 4 months ago (2016-08-04 19:35:56 UTC) #4
floitsch
LGTM. (although I'm pretty much asking you to rewrite everything...) https://codereview.chromium.org/2154933002/diff/1/sdk/lib/core/iterable.dart File sdk/lib/core/iterable.dart (right): https://codereview.chromium.org/2154933002/diff/1/sdk/lib/core/iterable.dart#newcode189 ...
4 years, 4 months ago (2016-08-05 09:38:40 UTC) #5
stanm
Addressed comment, but also ran dartfmt on the file by accident and it came up ...
4 years, 4 months ago (2016-08-05 16:48:31 UTC) #6
floitsch
Still LGTM. thanks.
4 years, 4 months ago (2016-08-05 17:29:13 UTC) #7
stanm
Committed patchset #2 (id:20001) manually as 0648e2622b2cc7d960993e51024108600be8c0b0 (presubmit successful).
4 years, 4 months ago (2016-08-05 17:34:47 UTC) #9
stanm
4 years, 4 months ago (2016-08-05 17:35:04 UTC) #10
Message was sent while issue was closed.
Thanks for the review.

Powered by Google App Engine
This is Rietveld 408576698