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

Issue 180074: code review 180074: Updated the XGB python script to generate semicolon-free (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 3 months ago by tor
Modified:
15 years, 2 months ago
Reviewers:
CC:
rsc, golang-dev
Visibility:
Public.

Description

Updated the XGB python script to generate semicolon-free source for xproto.go.

Patch Set 1 #

Patch Set 2 : code review 180074: Updated the XGB python script to generate semicolon-free #

Unified diffs Side-by-side diffs Delta from patch set Stats (+57 lines, -60 lines) Patch
M src/pkg/xgb/go_client.py View 15 chunks +57 lines, -60 lines 0 comments Download

Messages

Total messages: 6
tor
Hello rsc (cc: golang-dev@googlegroups.com), I'd like you to review the following change.
15 years, 3 months ago (2009-12-17 00:35:21 UTC) #1
rsc
looks good. should xproto.go be here too?
15 years, 3 months ago (2009-12-17 01:09:24 UTC) #2
tor
On 2009/12/17 01:09:24, rsc wrote: > looks good. > should xproto.go be here too? xproto.go ...
15 years, 3 months ago (2009-12-17 01:48:44 UTC) #3
rsc
> xproto.go hasn't changed, the earlier gofmt patches already > took away its semicolons. The ...
15 years, 2 months ago (2009-12-17 20:18:39 UTC) #4
rsc
LGTM
15 years, 2 months ago (2009-12-17 20:18:49 UTC) #5
rsc
15 years, 2 months ago (2009-12-17 20:19:44 UTC) #6
*** Submitted as http://code.google.com/p/go/source/detail?r=dd9bae5c7eac ***

Updated the XGB python script to generate semicolon-free
source for xproto.go.

R=rsc
CC=golang-dev
http://codereview.appspot.com/180074

Committer: Russ Cox <rsc@golang.org>
Sign in to reply to this message.

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