Age | Commit message (Collapse) | Author | Files | Lines |
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1007041
|
|
Enable package tests for Native Client build.
R=r
CC=golang-dev
http://codereview.appspot.com/957042
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/970042
|
|
Changed most of the functions in nat.go to methods on nat.
R=gri
CC=golang-dev
http://codereview.appspot.com/976041
Committer: Robert Griesemer <gri@golang.org>
|
|
Uid, Gid become int.
File size info becomes int64.
Times become int64.
R=rsc, cw
CC=golang-dev
http://codereview.appspot.com/968042
|
|
R=golang-dev, adg
CC=golang-dev
http://codereview.appspot.com/917042
Committer: Andrew Gerrand <adg@golang.org>
|
|
R=rsc, gri
CC=golang-dev
http://codereview.appspot.com/967041
Committer: Robert Griesemer <gri@golang.org>
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/953041
|
|
Fixes issue 733.
R=r
CC=golang-dev
http://codereview.appspot.com/958041
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/971041
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/970041
|
|
R=gri, eds
CC=golang-dev
http://codereview.appspot.com/947042
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/940041
Committer: Russ Cox <rsc@golang.org>
|
|
This gives about a 6% performance improvement to pidigits.
Thanks to Russ for the suggestion.
R=rsc, gri
CC=golang-dev
http://codereview.appspot.com/957041
Committer: Russ Cox <rsc@golang.org>
|
|
There was a bug in makeN that caused lots of unnecessary
allocations. Fixing this exposed a few bugs in other
functions which worked when makeN allocated a new slice, but
failed when it simply resized an existing slice. The result
is a pretty big performance improvement. When running
pidigits, here were the numbers I got on amd64:
Before this change:
pidigits 10000
gcc -O2 pidigits.c -lgmp 2.09u 0.02s 2.11r
gc pidigits 12.68u 0.04s 12.72r
gc_B pidigits 12.71u 0.03s 12.75r
After:
pidigits 10000
gcc -O2 pidigits.c -lgmp 2.09u 0.00s 2.10r
gc pidigits 6.82u 0.00s 6.85r
gc_B pidigits 6.55u 0.01s 6.59r
R=rsc, gri
CC=golang-dev
http://codereview.appspot.com/953042
Committer: Russ Cox <rsc@golang.org>
|
|
Due to page boundary rounding, the header would have
been loaded as part of the text segment already, but this
change placates the "paxctl" tool on so-called hardened
Linux distributions (as if normal distributions weren't already
hard enough to use).
R=r
CC=golang-dev
http://codereview.appspot.com/954041
|
|
This yields a pretty significant performance boost to pidigits and there are still some improvements to be made. Here are my numbers:
amd64 w/ bignum:
pidigits 10000
gcc -O2 pidigits.c -lgmp 2.10u 0.00s 2.10r
gc pidigits 22.92u 0.02s 22.97r
gc_B pidigits 22.62u 0.00s 22.65r
amd64 w/ big:
pidigits 10000
gcc -O2 pidigits.c -lgmp 2.09u 0.02s 2.11r
gc pidigits 12.68u 0.04s 12.72r
gc_B pidigits 12.71u 0.03s 12.75r
386 w/ bignum:
pidigits 10000
gcc -O2 pidigits.c -lgmp 2.09u 0.00s 2.09r
gc pidigits 44.30u 0.01s 44.35r
gc_B pidigits 44.29u 0.03s 44.35r
386 w/ big:
pidigits 10000
gcc -O2 pidigits.c -lgmp 2.10u 0.00s 2.10r
gc pidigits 22.70u 0.06s 22.79r
gc_B pidigits 22.80u 0.09s 22.91r
R=rsc, gri
CC=golang-dev
http://codereview.appspot.com/881050
Committer: Russ Cox <rsc@golang.org>
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/955041
|
|
to avoid confusion with real cgo
R=r
CC=golang-dev
http://codereview.appspot.com/904046
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/823048
|
|
R=rsc, gri
CC=golang-dev
http://codereview.appspot.com/929041
Committer: Robert Griesemer <gri@golang.org>
|
|
strings.Trim*Func functions.
R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/824051
Committer: Rob Pike <r@golang.org>
|
|
Fixes issue 729.
R=ken2
CC=golang-dev
http://codereview.appspot.com/875048
|
|
SETSID does return an errno - any reason why it has been done this
way in zsyscall_linux_* ? Otherwise it should be the same as darwin.
From SETSID(2) on my Linux box:
ERRORS
On error, -1 is returned, and errno is set.
Fixes issue 730
R=rsc
CC=golang-dev
http://codereview.appspot.com/878047
|
|
xml: add support for XML marshalling embedded structs.
R=rsc
CC=golang-dev
http://codereview.appspot.com/837042
Committer: Russ Cox <rsc@golang.org>
|
|
packStructValue was cutting off last byte of uint32
in _Dns_msg.Unpack, use packRR for rr types
R=rsc
CC=golang-dev
http://codereview.appspot.com/844048
Committer: Russ Cox <rsc@golang.org>
|
|
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
|
|
Fixes issue 675.
R=rsc, msolo
CC=golang-dev
http://codereview.appspot.com/882049
|
|
We were letting bignum decide, which caused problems with float literals with a leading 0.
R=gri
CC=golang-dev
http://codereview.appspot.com/816047
Committer: Robert Griesemer <gri@golang.org>
|
|
R=gri
CC=golang-dev
http://codereview.appspot.com/846050
Committer: Robert Griesemer <gri@golang.org>
|
|
R=golang-dev, gri
CC=golang-dev
http://codereview.appspot.com/908044
Committer: Robert Griesemer <gri@golang.org>
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/872045
|
|
$ godoc xml | grep Copy\(\)
func (c CharData) Copy() CharData
func (c Comment) Copy() Comment
func (d Directive) Copy() Directive
func (p ProcInst) Copy() ProcInst
func (e StartElement) Copy() StartElement
--------------------------------------------
$ godoc -src xml | grep Copy\(\)
func (c CharData) Copy() CharData
--------------------------------------------
$ godoc -src xml Copy
func (c CharData) Copy() CharData { return CharData(makeCopy(c)) }
--------------------------------------------
The command "godoc -src pkg_name" should output the interface of the named package, but it excludes all duplicate entries. Also the command "godoc -src pkg_name method_name" will output the source code only for one method even if there are more of them with the same name in the same package. This patch set fixes this issue.
R=gri
CC=golang-dev
http://codereview.appspot.com/883051
Committer: Robert Griesemer <gri@golang.org>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/912041
|
|
fmt.Printf("float32 %f\n", float32(1234.56789))
fmt.Printf("float64 %f\n", float64(1234.56789))
->
float32 1234.567871
float64 1234.567890
this is a snapshot. extended instruction support, corner cases
and fixes coming in subseuent cls.
R=rsc
CC=dpx, golang-dev
http://codereview.appspot.com/876045
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/840045
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/920041
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/854048
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/815044
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/851045
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/915041
Committer: Russ Cox <rsc@golang.org>
|
|
much rewriting and improving, but it's still experimental.
R=rsc
CC=golang-dev
http://codereview.appspot.com/875045
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/824049
|
|
R=gri, adg
CC=golang-dev, r, rsc
http://codereview.appspot.com/857048
Committer: Andrew Gerrand <adg@golang.org>
|
|
- don't log normal EOF
- fix ServeConn to block as documented
R=rsc, msolo
CC=golang-dev
http://codereview.appspot.com/886043
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/891050
|
|
R=rsc, r
CC=golang-dev
http://codereview.appspot.com/891048
Committer: Rob Pike <r@golang.org>
|
|
fmt.Printf("%b", int8(-1)) prints 64 ones instead of 8.
This happens only for signed integers (int8, in16 and int32). I guess it's because of the way the conversion between integer types works. From go spec: "Conversions between integer types. If the value is a signed quantity, it is sign extended to implicit infinite precision ....". And there are several conversions to int64 and uint64 in the fmt package. This pathch solves only half of the problem. On a 32 bit system, an fmt.Printf("%b", int(-1)) should still print 64 ones.
R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/891049
Committer: Rob Pike <r@golang.org>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/888045
Committer: Russ Cox <rsc@golang.org>
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/840043
|