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

Issue 2708343002: [dart:io][windows] Fix analyzer warning for last commit (Closed)

Created:
3 years, 10 months ago by zra
Modified:
3 years, 10 months ago
Reviewers:
Florian Schneider
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

[dart:io][windows] Fix analyzer warning for last commit Committed: https://github.com/dart-lang/sdk/commit/a18a8ae8ef486165bfbbddcffac669c0ad15140e

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M sdk/lib/io/stdio.dart View 1 chunk +3 lines, -1 line 2 comments Download

Messages

Total messages: 6 (2 generated)
zra
TBR
3 years, 10 months ago (2017-02-22 07:24:49 UTC) #2
zra
Committed patchset #1 (id:1) manually as a18a8ae8ef486165bfbbddcffac669c0ad15140e (presubmit successful).
3 years, 10 months ago (2017-02-22 07:26:03 UTC) #4
Florian Schneider
Lgtm w/ question https://codereview.chromium.org/2708343002/diff/1/sdk/lib/io/stdio.dart File sdk/lib/io/stdio.dart (right): https://codereview.chromium.org/2708343002/diff/1/sdk/lib/io/stdio.dart#newcode266 sdk/lib/io/stdio.dart:266: final IOSink _sink; Would changing this ...
3 years, 10 months ago (2017-02-22 15:56:55 UTC) #5
zra
3 years, 10 months ago (2017-02-22 17:35:14 UTC) #6
Message was sent while issue was closed.
https://codereview.chromium.org/2708343002/diff/1/sdk/lib/io/stdio.dart
File sdk/lib/io/stdio.dart (right):

https://codereview.chromium.org/2708343002/diff/1/sdk/lib/io/stdio.dart#newco...
sdk/lib/io/stdio.dart:266: final IOSink _sink;
On 2017/02/22 15:56:55, Florian Schneider wrote:
> Would changing this to _IOSindImpl fix it as well instead?

There's probably a smarter way to do all this, but right now, _sink is a
different implementation of IOSink (a Socket, I think) when invoked from
_StdSocketSink.

Powered by Google App Engine
This is Rietveld 408576698