summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2009-12-15gc: var x, ok = m[y]Russ Cox2-0/+40
Fixes issue 384. R=ken2 http://codereview.appspot.com/179061
2009-12-15Tweak test to work with both 6g and gccgo.Ian Lance Taylor1-3/+1
Pull the struct into a single line, since 6g reports the error on the last line of the struct definition and gccgo reports it on the first line. 6g: bug215.go:12: invalid recursive type A gccgo: bug215.go:12:6: error: invalid recursive type ‘A’ R=rsc http://codereview.appspot.com/180044
2009-12-15codereview: add golang-dev@googlegroups.comRuss Cox1-40/+33
automatically in "hg mail". also, avoid "empty list means all modified files in client" bug R=gri, cw CC=golang-dev http://codereview.appspot.com/174072
2009-12-15fix TODO: insert semicolons before any sequence of commentsRobert Griesemer2-45/+72
that introduce the newline (important for correct placement of comments with gofmt when parsing new syntax) R=rsc http://codereview.appspot.com/179055
2009-12-14xml: handle unexpected EOF while parsing and fix a bug in nameArvindh Rajesh Tamilmani2-27/+104
mustgetc reports unexpected EOF as SyntaxError. using mustgetc seems to be a better approach than letting the caller handle unexpected EOF every time. name: the second if statement should explicitly return ok==false. R=rsc http://codereview.appspot.com/174083 Committer: Russ Cox <rsc@golang.org>
2009-12-14runtime: in exitsyscall, avoid confusing garbage collectorRuss Cox1-1/+6
R=r CC=golang-dev http://codereview.appspot.com/178046
2009-12-14improved formatting of import declarations andRobert Griesemer6-7/+110
multi-line expressions with comments Fixes issue 414. R=rsc http://codereview.appspot.com/179047
2009-12-14crypto/sha256: new packageAndy Davis5-0/+333
R=rsc CC=golang-dev http://codereview.appspot.com/176062 Committer: Russ Cox <rsc@golang.org>
2009-12-15simpler fix for the negative rune problem, spotted seconds after submitting ↵Rob Pike1-19/+17
the previous fix. R=rsc http://codereview.appspot.com/178044
2009-12-15generate replacement rune when asked to encode a negative rune value.Rob Pike2-9/+26
Fixes issue 425. R=rsc http://codereview.appspot.com/178043
2009-12-14archive/tar: bug fixes.Christopher Wedgwood3-37/+167
1. If all data is exhausted using Read then a following Next will fail as if it saw EOF. (Test case added.) 2. Seeking isn't always possible (i.e. sockets and pipes). Fallback to read. (Test case added.) 3. Fix to readHeader (cleaner fix pointed out by rsc). (TestReader modified.) 4. When Read has consumed all the data, don't try to read 0 bytes from reader. In cases where tr.nb is zero we attempt to read zero bytes and thus never see an EOF (this is most easily seen when the 'tar source' is something like bytes.Buffer{} as opposed to os.File). 5. If write is used to the point of ErrWriteTooLong, allow additional file entries. 6. Make close work as expected. That is any further Write or WriteHeader attempts will result in ErrWriteAfterClose. Fixes issue 419. R=rsc, dsymonds1 http://codereview.appspot.com/162062 Committer: Russ Cox <rsc@golang.org>
2009-12-14don't show semicolons anymore with godocRobert Griesemer1-1/+1
R=rsc http://codereview.appspot.com/174078
2009-12-13A+C: two more namesRuss Cox2-0/+4
Andy Davis (individual CLA) Arvindh Rajesh Tamilmani (individual CLA) R=r http://codereview.appspot.com/176067
2009-12-14fix naked < as reported by pwil3058@gmail.comRob Pike2-2/+2
R=rsc CC=pwil3058 http://codereview.appspot.com/174087
2009-12-14When the buffer is empty, reset b.off to the beginning of the bufferRob Pike1-2/+20
to avoid growing unnecessarily. R=rsc CC=golang-dev http://codereview.appspot.com/176071
2009-12-13syscall: fix error return bug for 64-bit return on 32-bit platformChristopher Wedgwood5-29/+35
R=dho, rsc CC=r http://codereview.appspot.com/176058 Committer: Russ Cox <rsc@golang.org>
2009-12-138l: add support for PE output.Hector Chu7-8/+329
R=rsc http://codereview.appspot.com/166080 Committer: Russ Cox <rsc@golang.org>
2009-12-13clean.bash: stop if $GOROOT is not setChristopher Wedgwood1-0/+7
Doing rm -rf /pkg/.. blindly isn't nice. It could have unintended consequences. Secondly set bash to abort on (unexpected) errors. R=dho, rsc CC=golang-dev http://codereview.appspot.com/176056 Committer: Russ Cox <rsc@golang.org>
2009-12-13Ported godefs to Windows.Hector Chu3-67/+82
R=rsc http://codereview.appspot.com/164049 Committer: Russ Cox <rsc@golang.org>
2009-12-13gc: fix commentRuss Cox1-3/+2
R=gri CC=golang-dev http://codereview.appspot.com/174077
2009-12-14Update goyacc to new syntax, still with semicolons.Rob Pike2-67/+37
Update units.y too. Fixes issue 417. R=rsc CC=golang-dev http://codereview.appspot.com/176063
2009-12-12more on the optimizerKen Thompson2-67/+84
trying to get alizses to optimize R=rsc http://codereview.appspot.com/176061
2009-12-13fix bug for large counts: used a one-byte buffer.Rob Pike1-3/+2
R=rsc CC=golang-dev http://codereview.appspot.com/174082
2009-12-11Various cleanups:Robert Griesemer4-24/+18
- no need to replace comments for stand-alone blocks - always print string concatenations with interspersed "+" (remove option) - minor cleanups R=rsc http://codereview.appspot.com/174076
2009-12-11gc: semicolonsRuss Cox3-163/+101
Fixes issue 89. Fixes issue 92. Fixes issue 118. Fixes issue 182. Fixes issue 328. Fixes issue 340. R=ken2, ken3 CC=golang-dev http://codereview.appspot.com/172049
2009-12-11bug in 6g optimizerKen Thompson4-57/+57
8g still needs fixing R=rsc http://codereview.appspot.com/176057
2009-12-11parser changed to reflect new semicolon rulesRobert Griesemer2-229/+167
R=rsc http://codereview.appspot.com/175046 Committer: Robert Griesemer <gri@golang.org>
2009-12-11fix printer test for new syntaxRobert Griesemer9-105/+277
R=rsc http://codereview.appspot.com/175048
2009-12-11Remove GOBIN in PATH dependency; don't assume cwd is $GOROOT/srcDevon H. O'Dell21-111/+162
This change removes the necessity to have GOBIN in $PATH, and also doesn't assume that the build is being run from $GOROOT/src. This is a minimal set of necessary changes to get Go to build happily from the FreeBSD ports collection. R=rsc CC=golang-dev http://codereview.appspot.com/171044 Committer: Russ Cox <rsc@golang.org>
2009-12-11misc/bash: completion file for bashRuss Cox1-0/+6
suggested by Alex Ray <ajray@ncsu.edu> R=gri CC=golang-dev http://codereview.appspot.com/174063
2009-12-11encoding/binary: Add support for slices of arrays of fixed-size values.Maxim Ushakov2-6/+52
R=rsc CC=krasin http://codereview.appspot.com/167050 Committer: Russ Cox <rsc@golang.org>
2009-12-11Add a Makefile in place of all.bash. Update docs to reflectDevon H. O'Dell5-13/+19
this change. R=rsc CC=golang-dev http://codereview.appspot.com/171048 Committer: Russ Cox <rsc@golang.org>
2009-12-11syslog: new packageYves Junqueira5-3/+258
R=golang-dev, rsc http://codereview.appspot.com/157168 Committer: Russ Cox <rsc@golang.org>
2009-12-11mkbuiltin: generate builtin.c directlyYongjian Xu2-4/+2
R=golang-dev, rsc http://codereview.appspot.com/173041 Committer: Russ Cox <rsc@golang.org>
2009-12-11A+C:Russ Cox2-0/+5
Andrey Mirtchovski (individual CLA), Maxim Ushakov (Google), Yongjian Xu (individual CLA) R=gri http://codereview.appspot.com/174064
2009-12-11bytes, strings: allow -1 in Map to mean "drop this character".Kei Son6-26/+91
xml: drop invalid characters in attribute names when constructing struct field names. R=rsc CC=r http://codereview.appspot.com/157104 Committer: Russ Cox <rsc@golang.org>
2009-12-10- make make test working againRobert Griesemer1-15/+15
- some factoring for easier experimentation R=rsc http://codereview.appspot.com/174048
2009-12-10New flags for gofmt:Robert Griesemer4-11/+33
- oldparser parse old syntax (required semicolons) - oldprinter print old syntax (required semicolons) By default, these flags are enabled for now. Setting -oldparser=false has no effect until go/parser is changed to accept the new syntax. Enabled exp/parser in Makefile; update dependent exp/eval. R=rsc http://codereview.appspot.com/174051
2009-12-10rename exp/parser package to oldParserRobert Griesemer3-8/+13
to allow simultaneous import with the current go/parser R=rsc http://codereview.appspot.com/174053
2009-12-10Spec modified to reflect new semicolon rules.Robert Griesemer1-242/+273
R=rsc, r, iant, ken2 CC=golang-dev http://codereview.appspot.com/166066 Committer: Robert Griesemer <gri@golang.org>
2009-12-10implement NoSemis and NoStringConcat mode for go/printerRobert Griesemer2-5/+22
R=rsc http://codereview.appspot.com/174050
2009-12-10implemented InsertSemis mode for go/scannerRobert Griesemer2-20/+236
R=rsc http://codereview.appspot.com/175047
2009-12-10- unmodified copy of existing go/parser, not yet hooked upRobert Griesemer4-0/+2326
R=rsc CC=r http://codereview.appspot.com/175045
2009-12-10trailing comma's are not accepted with current syntaxRobert Griesemer1-2/+1
R=rsc, ken2 http://codereview.appspot.com/174047
2009-12-10test/fixedbug: drop obsolete bug125, bug166Russ Cox2-28/+0
R=gri CC=golang-dev http://codereview.appspot.com/173044
2009-12-10make test/ken safe for optional semisRobert Griesemer32-394/+188
R=rsc, ken2, ken3 http://codereview.appspot.com/174042
2009-12-10fix calling convention and make memmove restore the g and mKai Backman1-2/+10
registers. R=rsc http://codereview.appspot.com/166049
2009-12-10added SHUT_RD, SHOT_WR and xSHUT_RDWR to fix net compilation errorKai Backman1-0/+3
R=rsc http://codereview.appspot.com/165044
2009-12-10make test/fixedbugs save for optional semisRobert Griesemer8-17/+13
R=rsc http://codereview.appspot.com/173045
2009-12-09missed a couple of files in testRobert Griesemer2-18/+27
R=rsc, r http://codereview.appspot.com/172045