contribute.html

R=r, iant
CC=go-dev
http://go/go-review/1022007

require gofmt for all $GOROOT/src/.../*.go.

try harder to post messages to rietveld.

R=gri, r
CC=go-dev
http://go/go-review/1024006

crypto/x509: build fix for 32-bit

TBR=rsc

R=rsc
CC=go-dev
http://go/go-review/1024007

Fix add-back division test so that it triggers the add-back case on
both 64 and 32 bit platforms.

Also, uncomment some tests that were still commented out from
debugging.

R=gri, rsc
CC=go-dev
http://go/go-review/1026003

gofmt -w misc

R=rsc
http://go/go-review/1025004

make 5g executables run on android/arm hardware. change OABI
usage to EABI.

        go/test: passes 85% (296/347) on random android phone.

R=rsc
http://go/go-review/1024003

gofmt src/cmd/goyacc
(with this change: gofmt -l src/cmd/ | wc
is 0 0 0)

R=ken, rsc
http://go/go-review/1024005

gofmt'ed src/cmd

R=rsc
http://go/go-review/1024004

- fix elf.go manually so it is idempotent for gofmt
- with this change, `gofmt -l src/pkg | wc` is 0

R=rsc
http://go/go-review/1025002

missing piece gofmt'ed in reflect

R=r, rsc
http://go/go-review/1025001

gofmt'ed more stragglers

R=rsc
http://go/go-review/1019004

gofmt'ed big

R=agl
http://go/go-review/1022003

gofmt'ed various stragglers

R=rsc
http://go/go-review/1022002

gofmt'ed more stragglers
(now down to a handfull of files in pkg)

R=rsc
http://go/go-review/1019006

fix UTF-8 upload bug

TBR=r
http://go/go-review/1026001

gofmt the last outstanding files in src/pkg
- added a list of issues to printer/nodes.go

R=rsc
http://go/go-review/1024002

gofmt'ed missing regexp file
(one-line functions will be re-established once we have all
formatted uniformly)

R=r
http://go/go-review/1023004

manual fix for better gofmt result

R=r
http://go/go-review/1019005

- gofmt'ing of some stragglers, now with correct comment indentation
in special cases
- re-gofmt'ing of some files that are now improved

R=r, rsc
http://go/go-review/1023003

fix a comment formatting bug:
- this ensures better comment formatting in template.go and codec_test.go
- it affects only 2 files of all files that have been gofmt'ed already,
  see separate CL (it fixes the same issue in those files)

R=rsc
http://go/go-review/1023002

crypto/tls (part 6/5)

Results of running gofmt again.

R=rsc
CC=go-dev
http://go/go-review/1022006

fix broken link

R=rsc, iant
CC=go-dev
http://go/go-review/1019003

crypto/tls (part 5/5)

Make RSA and X509 build by using big. (This involves commenting out
key generation for now since I haven't written Miller-Rabin in big
yet.)

Add entries to the Makefile.

R=rsc
CC=go-dev
http://go/go-review/1022005

crypto/tls (part 4/5)

R=rsc
CC=go-dev
http://go/go-review/1019002

Cleaned up gccgo error messages require minor adjustment to
test.

const1.go:27: error: integer constant overflow
const1.go:29: error: integer constant overflow
const1.go:30: error: integer constant overflow
const1.go:31: error: integer constant overflow
const1.go:32: error: integer constant overflow
const1.go:33: error: integer constant overflow
const1.go:33: error: integer constant overflow
const1.go:34: error: integer constant overflow
const1.go:37: error: integer constant overflow
const1.go:38: error: integer constant overflow
const1.go:40: error: integer constant overflow
const1.go:41: error: integer constant overflow
const1.go:43: error: integer constant overflow
const1.go:44: error: integer constant overflow
const1.go:45: error: integer constant overflow
const1.go:48: error: integer constant overflow
const1.go:50: error: integer constant overflow
const1.go:51: error: integer constant overflow
const1.go:52: error: integer constant overflow
const1.go:53: error: integer constant overflow
const1.go:55: error: integer constant overflow
const1.go:56: error: division by zero
const1.go:59: error: floating point constant overflow
const1.go:61: error: floating point constant overflow
const1.go:62: error: division by zero
const1.go:47: error: integer constant overflow
const1.go:49: error: integer constant overflow
const1.go:60: error: floating point constant overflow
const1.go:68: error: argument 1 has incompatible type (cannot use type int8 as type int)
const1.go:69: error: argument 1 has incompatible type (cannot use type int8 as type int)
const1.go:70: error: argument 1 has incompatible type (cannot use type uint8 as type int)
const1.go:72: error: argument 1 has incompatible type (cannot use type float32 as type int)
const1.go:73: error: argument 1 has incompatible type (cannot use type float as type int)
const1.go:74: error: floating point constant truncated to integer
const1.go:76: error: argument 1 has incompatible type (cannot use type float64 as type int)
const1.go:77: error: argument 1 has incompatible type
const1.go:78: error: argument 1 has incompatible type

R=rsc
http://go/go-review/1022001

gofmt'ed parts of go

R=rsc
http://go/go-review/1023001

big: add Div, Mod, Exp, GcdExt and several other fixes.

R=gri, rsc
CC=go-dev
http://go/go-review/1017036

crypto/tls (part 3)

(With hindsight, record_process might have been designed wrong, but it
works for now. It'll get redrawn when client support is added.)

R=rsc
CC=r
http://go/go-review/1018032

gofmt-ify io, json, runtime, encoding

R=rsc
http://go/go-review/1017056

gofmt'ed godefs

R=rsc
http://go/go-review/1020001

gofmt'ed debug
(excluding debug/gosym/symtab.go which has a small issue)

R=rsc
http://go/go-review/1019001

gofmt'ed 6l 8l cgo

R=r
http://go/go-review/1020002

gofmt-ify strings, template

R=r
http://go/go-review/1018064

add information comparing the compiler implementations to the installation document

R=rsc
CC=go-dev
http://go/go-review/1018063

gofmt-ify gob
(the one-line struct types used in composite literals will become
one line again in another cleanup round; don't worry about them now)

R=r
http://go/go-review/1016056

codereview: new commands
* clpatch
* download
* submit, on behalf of clpatch

stir hgpatch to fix a few bugs

R=r
CC=go-dev
http://go/go-review/1016051

gofmt-ify 4s, iterable

R=rsc
http://go/go-review/1016055

gofmt-ify draw

R=rsc
http://go/go-review/1017055

gofmt-ify eval

R=rsc
http://go/go-review/1016054

gofmt-ify ogle

R=rsc
http://go/go-review/1016053

gofmt-ify nacl

R=rsc
http://go/go-review/1018062

gofmt-ify reflect
- the single line structs can be fixed in another round

R=rsc
http://go/go-review/1016052

2nd attempt: no noIndent for string lists
- slightly better output in general
- extra indentation where we could do without
  (however that seems better for now that not having the indentation
  where it is needed)
- no information is lost, so a future better approach can fix these
  places up again

R=rsc
http://go/go-review/1017050

add a place-holder link for the hour-long video

R=rsc
CC=go-dev
http://go/go-review/1017054

update the text on the landing page.

R=agl, rsc
CC=go-dev
http://go/go-review/1017053

forgot a piece of debugging
in previous change

R=rsc
http://go/go-review/1017052

same speedup as 6l

R=rsc
http://go/go-review/1016050

godoc: style update.

R=r, rsc
CC=go-dev
http://go/go-review/1017051

gofmt-ify syscall
(replacement for CL 1018053)

R=r
http://go/go-review/1017047

gofmt-ify template, time, unsafe, flag, fmt
(replacement for CLs 1017039, 1017041, 1017040, 1018054)

R=r
http://go/go-review/1018060

new command hgpatch, for use by codereview extension

R=r
http://go/go-review/1018059

gofmt-ify math, expvar, exp/spacewar

R=rsc
http://go/go-review/1018061

gofmt-ify parts of xml

R=rsc
http://go/go-review/1017049

gofmt-ify encoding

R=rsc
http://go/go-review/1017048

add -l flag to hg gofmt.
don't apply gofmt to non-go files during upload check.

R=r
http://go/go-review/1016048

add "hg gofmt" to run all modified files through gofmt.

add check before Upload, for now disabled by default,
that files do not change when run through gofmt.
enable by adding

[codereview]
force_gofmt = True

to .hgrc or .hg/hgrc.

add doc strings for a few more commands.
rename codereview-login to code-login
to make module doc visible.

R=r
CC=gri
http://go/go-review/1018056

gofmt-ify sort, sync

R=rsc
http://go/go-review/1018057

gofmt-ify strconv
(and remove an empty line at the end of a file)

R=rsc
http://go/go-review/1017046

gofmt-ify net

R=rsc
http://go/go-review/1017045

gofmt-ify image

R=rsc
http://go/go-review/1017044

gofmt-ify hash, http
(gofmt will be able to re-align map entries as in http nicely,
eventually)

R=rsc
http://go/go-review/1018055

gofmt-ify tabwriter

R=rsc
http://go/go-review/1017042

more comment formatting:
- preserve (some) indentation of comment text for /*-style comments
  even if the first comment line does not contain any text that might
  suggest the "correct" indentation
- enabled because otherwise existing larger comments get re-formatted
  (this will not introduce a lot of changes since comments of this
  kind - until now - were not changed with respect to indentation)

R=rsc
http://go/go-review/1016047

bumper for video

R=rsc
http://go/go-review/1017038

gofmt-ify unicode

R=r
http://go/go-review/1018051

fixing (badly) linux_arm syscall.

R=rsc
http://go/go-review/1017037

- add a blank before the opening { for multi-line composites
(request by r)
- feature temporarily disabled

R=rsc
CC=r
http://go/go-review/1018052

gofmt-ify os

R=r
http://go/go-review/1017034

speed up 6l
hopefully no change

R=rsc
http://go/go-review/1017035

package patch

R=r
http://go/go-review/1018043

add a final section with a complete example

R=rsc
CC=go-dev
http://go/go-review/1017033

gofmt-ify compress

R=rsc
http://go/go-review/1016045

gofmt-ify ebnf

R=r
http://go/go-review/1018050

gofmt-ify bufio

R=rsc
http://go/go-review/1018049

gofmt-ify asn1

R=rsc
http://go/go-review/1018047

gofmt-ify src/pkg/archive

R=rsc
http://go/go-review/1018046

gofmt-ify crypto

R=agl
CC=rsc
http://go/go-review/1017032

gofmt-ify src/pkg/go (excluding printer directory due to pending CL,
and parser.go and scanner_test.go which have minor formatting issues)

R=rsc
http://go/go-review/1016042

rename testfiles from .go -> .orig so that they won't be
picked up by gofmt (they should not be modified) or godoc
(testdata should not be considered a potential package
directory)

R=rsc
http://go/go-review/1018045

gofmt-ify bytes

R=r
http://go/go-review/1016044

better placement of /*-style comments interspersed with code on one line

R=rsc
http://go/go-review/1017030

bytes.SplitAfter and strings.SplitAfter
most common usage is:

lines := strings.SplitAfter(text, "\n", 0)

R=r
http://go/go-review/1018042

doc doc doc

R=r
http://go/go-review/1016039

avoid infinite recursion in matcher.

after sync (or sync --local), clean up repository:
* look for and close CLs submitted on our behalf
* remove unmodified files from CLs
* warn about empty CLs

R=r
http://go/go-review/1017029

panic can't print interfaces well so use  err.String()

R=rsc
http://go/go-review/1016041

- always format blocks with closing } on a new line, even if empty
- at the moment this also affects single-line function declarations
  because we have disabled them at the moment (but not single-line
  function literals)

R=rsc
http://go/go-review/1016040

stylistic issues in a couple of cmd docs

R=rsc
http://go/go-review/1017028

document godoc and gofmt

R=rsc
http://go/go-review/1018036

remove reference to /home/r

R=rsc
http://go/go-review/1018039

- complete html-escaping also in printer.go

R=rsc
http://go/go-review/1017027

- respect source line breaks in grouped declarations
- made ast.Spec nodes implement Node interface
- added extra test cases

R=rsc
http://go/go-review/1016038

fix bufio test case

R=rsc
http://go/go-review/1018038

remove "em" unit from td width specification

R=rsc
http://go/go-review/1018037

two more regexp tweaks

TBR=r
http://go/go-review/1016037

allow "no such host" and also "no answer from server";
seems to help on snow leopard.

TBR=r
http://go/go-review/1018035

various tweaks to code review.
main one is to check at submit time that
user name being used in checkin message
is listed in the CONTRIBUTORS file.
this should catch misconfigurations.

another is to cut the @domain part
from the R= and CC= lines on checkin
messages, so that cc'ing someone on
a change does not mean their email
address is recorded for all time.

R=r
CC=go-dev
http://go/go-review/1016036

document gotest

R=rsc
CC=go-dev
http://go/go-review/1017026

add " and ' to list of html-escaped chars

R=rsc
http://go/go-review/1017025

add goyacc to the build

R=rsc
CC=go-dev
http://go/go-review/1018034

make goyacc and units.y work again (minor tweaks).
document goyacc a little.

R=rsc
CC=go-dev
http://go/go-review/1017024