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

Issue 4634042: code review 4634042: [release-branch.r57] docs/GoCourseDay1.pdf: fix error i... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 8 months ago by rsc
Modified:
9 years, 8 months ago
Reviewers:
r2
CC:
r, golang-dev
Visibility:
Public.

Description

[release-branch.r57] docs/GoCourseDay1.pdf: fix error in operator table. ««« CL 4637041 / df607ef238c9 docs/GoCourseDay1.pdf: fix error in operator table. Communications op was listed as a binary; it isn't any more. R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/4637041 »»»

Patch Set 1 #

Patch Set 2 : diff -r 7b779072ec76 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -0 lines) Patch
M doc/GoCourseDay1.pdf View 1 Binary file 0 comments Download

Messages

Total messages: 4
rsc
Hello r (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://go.googlecode.com/hg/
9 years, 8 months ago (2011-06-16 06:04:54 UTC) #1
rsc
*** Submitted as http://code.google.com/p/go/source/detail?r=9d7967223815 *** [release-branch.r57] docs/GoCourseDay1.pdf: fix error in operator table. ««« CL 4637041 ...
9 years, 8 months ago (2011-06-16 06:04:58 UTC) #2
r2
LGTM is it r57 or r57.1 or r57.2? -rob
9 years, 8 months ago (2011-06-16 06:12:25 UTC) #3
rsc
9 years, 8 months ago (2011-06-16 06:14:30 UTC) #4
> is it r57 or r57.1 or r57.2?

those are all points along release-branch.r57
(named in the subject).  r57.2 is the one we
just tagged (after this CL).
Sign in to reply to this message.

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