code review 4975046: weekly.2011-09-01

weekly.2011-09-01

[nigeltao, 2011-09-01 06:44:51]

Hello r@golang.org (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/

[adg, 2011-09-01 06:55:32]

LGTM

http://codereview.appspot.com/4975046/diff/4001/doc/devel/weekly.html
File doc/devel/weekly.html (right):

http://codereview.appspot.com/4975046/diff/4001/doc/devel/weekly.html#newcode22
doc/devel/weekly.html:22: The gc compiler now does escape analysis, which
improves program performance by placing variables on the call stack instead of
the heap when it is safe to do so.
wrap at 80 cols

[r, 2011-09-01 06:58:21]

http://codereview.appspot.com/4975046/diff/4001/doc/devel/weekly.html
File doc/devel/weekly.html (right):

http://codereview.appspot.com/4975046/diff/4001/doc/devel/weekly.html#newcode24
doc/devel/weekly.html:24: Despite the long list of changes, there are no major
API changes.
you don't say anything about container/vector

[nigeltao, 2011-09-01 07:05:26]

On 1 September 2011 16:58,  <r@golang.org> wrote:
> you don't say anything about container/vector

PTAL.

[r, 2011-09-01 07:07:07]

LGTM

[nigeltao, 2011-09-01 07:09:03]

*** Submitted as http://code.google.com/p/go/source/detail?r=c934f6f5fe8b ***

weekly.2011-09-01

R=r, adg
CC=golang-dev
http://codereview.appspot.com/4975046