Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
gob: substitute slice for map
R=r
CC=golang-dev
http://codereview.appspot.com/1699045
|
|
update other code to match.
R=r
CC=golang-dev
http://codereview.appspot.com/1680044
|
|
R=gri
CC=golang-dev
http://codereview.appspot.com/1620042
|
|
Fixes issue 773.
R=adg
CC=golang-dev
http://codereview.appspot.com/1120042
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1173041
Committer: Russ Cox <rsc@golang.org>
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/943047
|
|
R=r, cw
CC=golang-dev
http://codereview.appspot.com/952041
|
|
This matches the old JSON package behavior.
All lowercase names are not as standard as I believed,
and it seems less surprising to need to write
type T struct { Field string "field" }
to get lower case (behavior after this CL) than it does to need
to write
type T struct { Field string "Field" }
to preserve the case (behavior before this CL).
Also test and fix unmarshal into non-nil interface
value or pointer.
Fixes issue 744.
R=r
CC=golang-dev
http://codereview.appspot.com/1013041
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/953041
|
|
This is the first of probably four separate CLs
for the new implementation of the json package.
The scanner is the core of the new implementation.
The other CLs would be the new decoder,
the new encoder, and support for JSON streams.
R=r
CC=golang-dev
http://codereview.appspot.com/802051
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/888045
Committer: Russ Cox <rsc@golang.org>
|
|
R=r, gri
CC=golang-dev
http://codereview.appspot.com/872041
|
|
Fixes issue 661
R=r, rsc, skorobo
CC=golang-dev
http://codereview.appspot.com/576042
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/217047
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc, imkrasin
CC=golang-dev
http://codereview.appspot.com/179125
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
CC=golang-dev, jack.palevich
http://codereview.appspot.com/183109
|
|
R=gri
CC=golang-dev
http://codereview.appspot.com/179128
|
|
parsing and printing to new syntax.
Use -oldparser to parse the old syntax,
use -oldprinter to print the old syntax.
2) Change default gofmt formatting settings
to use tabs for indentation only and to use
spaces for alignment. This will make the code
alignment insensitive to an editor's tabwidth.
Use -spaces=false to use tabs for alignment.
3) Manually changed src/exp/parser/parser_test.go
so that it doesn't try to parse the parser's
source files using the old syntax (they have
new syntax now).
4) gofmt -w src misc test/bench
3rd set of files.
R=rsc
CC=golang-dev
http://codereview.appspot.com/180048
|
|
R=rsc
http://codereview.appspot.com/172041
|
|
Fixes issue 400.
R=golang-dev, rsc
http://codereview.appspot.com/167058
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
http://codereview.appspot.com/164071
|
|
This patch adds an ability to convert JSON-encoded data into
a hierarchy of Go's native data types.
R=rsc
CC=golang-dev
http://codereview.appspot.com/161060
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 272.
R=rsc
http://codereview.appspot.com/161061
Committer: Russ Cox <rsc@golang.org>
|
|
- removed New(len int) in favor of new(Vector).Resize(len, cap)
- removed Init(len int) in favor of Resize(len, cap)
- runs all.bash
Fixes issue 294.
R=rsc, r, r1
http://codereview.appspot.com/157143
|
|
some error-related tests
R=rsc
http://codereview.appspot.com/157151
Committer: Russ Cox <rsc@golang.org>
|
|
R=r, r1
http://codereview.appspot.com/157146
|
|
the bash scripts and makefiles for building go didn't take into account
the fact $GOROOT / $GOBIN could both be directories containing whitespaces,
and was not possible to build it in such a situation.
this commit adjusts the various makefiles/scripts to make it aware of that
possibility, and now it builds successfully when using a path with whitespaces
as well.
Fixes issue 115.
R=rsc, dsymonds1
http://codereview.appspot.com/157067
Committer: Russ Cox <rsc@golang.org>
|
|
R=r, gri
CC=golang-dev
http://codereview.appspot.com/156115
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/157068
Committer: Russ Cox <rsc@golang.org>
|
|
I screwed up and didn't write one of the code review changes to disk
before submitting.
TBR=rsc
R=rsc
http://codereview.appspot.com/154122
|
|
Fixies issue 114.
R=rsc
CC=golang-dev
http://codereview.appspot.com/154121
|
|
Fixes issue 147.
R=rsc
CC=golang-dev
http://codereview.appspot.com/152123
|
|
Fixes issue 73.
R=rsc
CC=golang-dev
http://codereview.appspot.com/154072
|
|
rsc's algorithm
- applied gofmt -w misc src
- partial CL (last chunk)
R=rsc, r
http://go/go-review/1024041
|
|
R=rsc, r
http://go/go-review/1025029
|
|
- enabled for function declarations (not just function literals)
- applied gofmt -w $GOROOT/src
(look for instance at src/pkg/debug/elf/elf.go)
R=r, rsc
CC=go-dev
http://go/go-review/1026006
|
|
R=rsc
http://go/go-review/1017056
|
|
now that reflection supports it.
R=r
http://go/go-review/1015008
|
|
R=gri
DELTA=1359 (138 added, 32 deleted, 1189 changed)
OCL=35408
CL=35420
|