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

Issue 5677063: code review 5677063: exp/winfsnotify: remove reference to _test (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 1 month ago by r
Modified:
13 years, 1 month ago
Reviewers:
CC:
golang-dev, dsymonds, r2
Visibility:
Public.

Description

exp/winfsnotify: remove reference to _test Updates issue 2573.

Patch Set 1 #

Patch Set 2 : diff -r 1a03fdfef2ab https://code.google.com/p/go #

Total comments: 1

Patch Set 3 : diff -r 1a03fdfef2ab https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Patch
M src/pkg/exp/winfsnotify/winfsnotify_test.go View 1 2 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 6
r
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go
13 years, 1 month ago (2012-02-16 04:19:33 UTC) #1
dsymonds
Does it need to watch a test-specific subdir of tmpdir? Won't just watching tmpdir run ...
13 years, 1 month ago (2012-02-16 04:24:43 UTC) #2
r
Hello golang-dev@googlegroups.com, dsymonds@golang.org (cc: golang-dev@googlegroups.com), Please take another look.
13 years, 1 month ago (2012-02-16 04:27:19 UTC) #3
dsymonds
LGTM http://codereview.appspot.com/5677063/diff/4/src/pkg/exp/winfsnotify/winfsnotify_test.go File src/pkg/exp/winfsnotify/winfsnotify_test.go (right): http://codereview.appspot.com/5677063/diff/4/src/pkg/exp/winfsnotify/winfsnotify_test.go#newcode118 src/pkg/exp/winfsnotify/winfsnotify_test.go:118: dir, err := ioutil.TempDir("", "inotify") perhaps s/inotify/win&/ to ...
13 years, 1 month ago (2012-02-16 04:28:36 UTC) #4
r2
On Feb 16, 2012, at 3:28 PM, dsymonds@golang.org wrote: > LGTM > > > http://codereview.appspot.com/5677063/diff/4/src/pkg/exp/winfsnotify/winfsnotify_test.go ...
13 years, 1 month ago (2012-02-16 04:34:05 UTC) #5
r
13 years, 1 month ago (2012-02-16 04:34:32 UTC) #6
*** Submitted as 87cc9108a3b9 ***

exp/winfsnotify: remove reference to _test
Updates issue 2573.

R=golang-dev, dsymonds, r
CC=golang-dev
http://codereview.appspot.com/5677063
Sign in to reply to this message.

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