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

Issue 7577050: code review 7577050: doc/go1.1.html: performance (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years ago by r
Modified:
12 years ago
Reviewers:
CC:
golang-dev, remyoudompheng, bradfitz, adg1
Visibility:
Public.

Description

doc/go1.1.html: performance An overview is all that's required.

Patch Set 1 #

Total comments: 1

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

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

Total comments: 1

Patch Set 4 : diff -r cfcccdab127e https://code.google.com/p/go #

Patch Set 5 : diff -r ec0937aebe0a https://code.google.com/p/go/ #

Patch Set 6 : diff -r ec0937aebe0a https://code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -6 lines) Patch
M doc/go1.1.html View 1 2 3 4 2 chunks +21 lines, -6 lines 0 comments Download

Messages

Total messages: 7
r
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go
12 years ago (2013-03-22 23:55:51 UTC) #1
remyoudompheng
https://codereview.appspot.com/7577050/diff/1/doc/go1.1.html File doc/go1.1.html (right): https://codereview.appspot.com/7577050/diff/1/doc/go1.1.html#newcode335 doc/go1.1.html:335: <li>The gc compilers now in-line some small functions, including ...
12 years ago (2013-03-23 00:11:16 UTC) #2
r
Hello golang-dev@googlegroups.com, remyoudompheng@gmail.com (cc: golang-dev@googlegroups.com), Please take another look.
12 years ago (2013-03-23 00:14:58 UTC) #3
bradfitz
https://codereview.appspot.com/7577050/diff/8001/doc/go1.1.html File doc/go1.1.html (right): https://codereview.appspot.com/7577050/diff/8001/doc/go1.1.html#newcode344 doc/go1.1.html:344: <li>The garbabe collector is also more precise, which costs ...
12 years ago (2013-03-23 00:22:50 UTC) #4
r
If there wasn't so much modem noise in HTML it would be easier to find ...
12 years ago (2013-03-23 00:25:12 UTC) #5
adg1
LGTM
12 years ago (2013-03-23 15:07:32 UTC) #6
r
12 years ago (2013-03-23 15:30:04 UTC) #7
*** Submitted as https://code.google.com/p/go/source/detail?r=1405be4b6fc4 ***

doc/go1.1.html: performance
An overview is all that's required.

R=golang-dev, remyoudompheng, bradfitz, adg
CC=golang-dev
https://codereview.appspot.com/7577050
Sign in to reply to this message.

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