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

Issue 10703096: [IO] Improve documentation for dart:io Socket.onWrite. (Closed)

Created:
8 years, 5 months ago by Bill Hesse
Modified:
8 years, 5 months ago
Reviewers:
Søren Gjesse
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

[IO] Improve documentation for dart:io Socket.onWrite. BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=9428

Patch Set 1 #

Total comments: 2

Patch Set 2 : address comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M runtime/bin/socket.dart View 1 1 chunk +4 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Bill Hesse
8 years, 5 months ago (2012-07-05 15:53:19 UTC) #1
Søren Gjesse
lgtm http://codereview.chromium.org/10703096/diff/1/runtime/bin/socket.dart File runtime/bin/socket.dart (right): http://codereview.chromium.org/10703096/diff/1/runtime/bin/socket.dart#newcode78 runtime/bin/socket.dart:78: * writing. Then the handler is reset to ...
8 years, 5 months ago (2012-07-06 04:02:11 UTC) #2
Bill Hesse
8 years, 5 months ago (2012-07-06 08:41:24 UTC) #3
http://codereview.chromium.org/10703096/diff/1/runtime/bin/socket.dart
File runtime/bin/socket.dart (right):

http://codereview.chromium.org/10703096/diff/1/runtime/bin/socket.dart#newcode78
runtime/bin/socket.dart:78: * writing. Then the handler is reset to null. This
handler is used when
On 2012/07/06 04:02:11, Søren Gjesse wrote:
> is reset -> is automatically reset

Done.

Powered by Google App Engine
This is Rietveld 408576698