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

Issue 1604903006: Add clearer rules for implementations. (Closed)

Created:
4 years, 11 months ago by nweiz
Modified:
4 years, 11 months ago
Reviewers:
Bob Nystrom
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/stream_channel.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Add clearer rules for implementations. This helps implementations know how to handle corner cases, and helps users know what behavior they can rely on. R=rnystrom@google.com Committed: https://github.com/dart-lang/stream_channel/commit/c8515f0d6461faeb5c2e8a9cbd58d1924809ae86

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -5 lines) Patch
M lib/stream_channel.dart View 1 chunk +42 lines, -5 lines 0 comments Download

Messages

Total messages: 7 (1 generated)
nweiz
4 years, 11 months ago (2016-01-21 01:08:11 UTC) #1
gzoechi
On 2016/01/21 01:08:11, nweiz wrote: I had to read the sentence containing `[sink]—canceling` a few ...
4 years, 11 months ago (2016-01-21 12:36:55 UTC) #2
Bob Nystrom
lgtm
4 years, 11 months ago (2016-01-21 20:38:22 UTC) #3
nweiz
On 2016/01/21 12:36:55, gzoechi wrote: > On 2016/01/21 01:08:11, nweiz wrote: > > I had ...
4 years, 11 months ago (2016-01-21 23:28:58 UTC) #4
nweiz
Committed patchset #1 (id:1) manually as c8515f0d6461faeb5c2e8a9cbd58d1924809ae86 (presubmit successful).
4 years, 11 months ago (2016-01-21 23:29:19 UTC) #6
Bob Nystrom
4 years, 11 months ago (2016-01-21 23:41:47 UTC) #7
Message was sent while issue was closed.
On 2016/01/21 23:28:58, nweiz wrote:
> On 2016/01/21 12:36:55, gzoechi wrote:
> > On 2016/01/21 01:08:11, nweiz wrote:
> > 
> > I had to read the sentence containing `[sink]—canceling` a few times and
> > wondered what this means.
> > Maybe this some bias of my German native tongue, but I think `-` should be
> > change dot ` - ` (added spaces) or `, `.
> 
> I've considered changing it, but I think I'm going to keep it as-is. This is a
> pretty standard use of the em dash (although apparently in the UK it's more
> usual to add space around it), and it's the most appropriate punctuation to
> indicate that the two sentences are separate but closely related.

For what it's worth, I found the em dash hard to read there too. I would have
considered just using a period or rewording the sentence.

Powered by Google App Engine
This is Rietveld 408576698