Age | Commit message (Collapse) | Author | Files | Lines |
|
- remove use of implicit string concatenation
- these appear to be the only files correctly compiling under test
that used implicit string concatenation
R=rsc
http://codereview.appspot.com/172043
|
|
R=r
http://codereview.appspot.com/165083
Committer: Russ Cox <rsc@golang.org>
|
|
shootout.alioth.debian.org .
it's now there: http://shootout.alioth.debian.org/u32q/benchmark.php?test=chameneosredux&lang=all&box=1!
R=r, rsc
CC=golang-dev
http://codereview.appspot.com/167043
Committer: Russ Cox <rsc@golang.org>
|
|
1.9s gcc reverse-complement.c
reverse-complement.go
4.5s / 3.5s original, with/without bounds checks
3.5s / 3.3s bounds check reduction
3.3s / 2.8s smarter garbage collector
2.6s / 2.3s assembler bytes.IndexByte
2.5s / 2.1s even smarter garbage collector
2.3s / 2.1s fix optimizer unnecessary spill bug
2.0s / 1.9s change loop to range (this CL)
R=r
http://codereview.appspot.com/166072
|
|
R=r
http://codereview.appspot.com/165065
|
|
add README explaining how to try the
web demos.
Fixes issue 339.
R=r
CC=barry.d.silverman, bss, vadim
http://codereview.appspot.com/165057
|
|
R=ken2
http://codereview.appspot.com/165055
|
|
Fixes issue 238.
R=ken2
http://codereview.appspot.com/163098
|
|
Fixes issue 245.
R=ken2
http://codereview.appspot.com/164094
|
|
Fixes bug 292.
R=ken2
http://codereview.appspot.com/164093
|
|
Fixes issue 364.
R=ken2
http://codereview.appspot.com/164092
|
|
Fixes issue 370.
R=ken2
http://codereview.appspot.com/163097
|
|
this breaks the dependency of package io on package bytes.
R=rsc
CC=golang-dev
http://codereview.appspot.com/163085
|
|
unnamed, not require -n, in line with other
entries.
R=rsc
CC=r
http://codereview.appspot.com/162061
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/164047
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 288.
R=ken2
http://codereview.appspot.com/157162
|
|
R=rsc
CC=golang-dev, rog
http://codereview.appspot.com/160054
|
|
- 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
|
|
based on suggestion from arvindht@gmail.com
R=r, rsc, r1
http://codereview.appspot.com/157091
Committer: Russ Cox <rsc@golang.org>
|
|
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>
|
|
except chameneosredux which i know is being edited
require gofmt for test/bench
R=r
http://codereview.appspot.com/157110
|
|
R=ken2
http://codereview.appspot.com/157114
|
|
* add runtime sliceslice1 for x[lo:]
* remove runtime arraytoslice, rewriting &arr into arr[0:len(arr)].
* port cgen_inline into 8g, 5g.
* use native memmove in maps
R=ken2
http://codereview.appspot.com/157106
|
|
* reverse-complement: port C algorithm to Go
saves 30% on my MacBook Pro and makes it a fairer comparison.
* test reverse-complement with and without GC (another 15%)
* revise timing.sh to work on more systems
* avoid two glibcisms in fasta.c
R=r
http://codereview.appspot.com/156110
|
|
R=r, rsc
http://codereview.appspot.com/157085
|
|
- actually test the results
R=rsc, ken2
CC=golang-dev
http://codereview.appspot.com/156094
|
|
R=rsc, ken2
CC=golang-dev
http://codereview.appspot.com/156084
|
|
replace all calls with calls to copy
use copy in regexp and bytes.Buffer
R=rsc
CC=golang-dev
http://codereview.appspot.com/157073
|
|
R=rsc
http://codereview.appspot.com/154172
|
|
Fixes issue 167.
R=ken2
http://codereview.appspot.com/155062
|
|
* check for struct literal assignment to private fields.
* record, fix crash involving parallel map assignment.
* avoid infinite recursion in exportassignok.
* make floating point bounds check precise.
* avoid crash on invalid receiver.
* add context to error about implicit assignment.
Fixes issue 86.
Fixes issue 88.
Fixes issue 158.
Fixes issue 174.
Fixes issue 201.
Fixes issue 204.
R=ken2
http://codereview.appspot.com/154144
|
|
R=rsc
http://codereview.appspot.com/154097
|
|
versions of static checks in receiver1.go
R=r
http://codereview.appspot.com/155045
|
|
This does still contain some FreeBSD-specific bits, but
it's a pain to do partial diffs.
R=rsc
http://codereview.appspot.com/152138
Committer: Russ Cox <rsc@golang.org>
|
|
performance hit of about 20% but more intuitive results for submatches.
we need a good regexp package at some point.
Fixes issue 110.
R=rsc
CC=golang-dev
http://codereview.appspot.com/152131
|
|
use bash builtin time instead of
assuming /usr/bin/time exists.
R=r
http://codereview.appspot.com/152124
|
|
Fixes issue 116.
R=r
http://codereview.appspot.com/152106
|
|
too hard to make it the same everywhere.
still checking for non-zero exit status though.
disable core files while running tests
R=r
http://go/go-review/1026037
|
|
R=ken
http://go/go-review/1026032
|
|
adjust an inconsistency in reflect/type.go's package comment
R=rsc
http://go/go-review/1026029
|
|
usage to EABI.
go/test: passes 85% (296/347) on random android phone.
R=rsc
http://go/go-review/1024003
|
|
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
|
|
tests are processed, and thus the run.out output. The default
LANG on Fedora 10, en_US.utf8, causes the tests to be sorted
differently--e.g., arrayindex1.go and arrayindex.go are sorted
in the opposite order.
R=r, rsc
http://go/go-review/1018022
|
|
type. This is OK in a conversion but not in an assignment.
R=rsc
http://go/go-review/1016024
|
|
global regexps.
R=rsc, gri
CC=go-dev
http://go/go-review/1016025
|
|
runtime.go:19:10: error: reference to undefined identifier ‘runtime.printbool’
R=rsc
http://go/go-review/1018018
|
|
R=rsc
http://go/go-review/1017012
|
|
fix nil pointer check in 6g.
was dereferencing after the ADD;
dereference before instead.
R=ken@golang.org
CC=iant
http://go/go-review/1016022
|
|
go/test: passes 99% (343/347)
R=rsc
http://go/go-review/1016004
|
|
R=r
CC=go-dev
http://go/go-review/1016006
|