Age | Commit message (Collapse) | Author | Files | Lines |
|
stab in the dark but plausible: the kernel does try to
return time zone information.
http://fxr.watson.org/fxr/source/kern/kern_time.c?v=FREEBSD8#L421
R=iant
CC=golang-dev
http://codereview.appspot.com/206053
|
|
finalize chan, to free OS X semaphore inside Lock.
os: finalize File, to close fd.
Fixes issue 503.
R=ken2
CC=golang-dev
http://codereview.appspot.com/204065
|
|
add simple garbage collection benchmark.
R=iant
CC=golang-dev
http://codereview.appspot.com/204053
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/204061
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/203053
|
|
Allows binary to run on some Linux system.
Fix for issue 365.
R=rsc
CC=golang-dev
http://codereview.appspot.com/199096
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/199082
Committer: Russ Cox <rsc@golang.org>
|
|
R=r, cw
CC=golang-dev
http://codereview.appspot.com/198085
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/199042
|
|
No longer a distinct type; now a property of func types.
R=ken2
CC=golang-dev
http://codereview.appspot.com/197042
|
|
Fixes issue 219.
R=ken2, r
CC=golang-dev
http://codereview.appspot.com/194097
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/193104
|
|
(eliminate assumption of package global name space,
make code easier to move between packages).
R=r
CC=golang-dev
http://codereview.appspot.com/194072
|
|
bonus: type switch now detects multiple uses of identical interface types.
bonus: interface types are now order-independent, following the spec.
R=ken2
CC=golang-dev
http://codereview.appspot.com/194053
|
|
detect compilation of special package runtime with
compiler flag instead of package name.
R=ken2
CC=golang-dev
http://codereview.appspot.com/193080
|
|
need better management of mach semaphores eventually
but this avoids allocating them for uncontended Locks.
R=r
CC=agl1, golang-dev
http://codereview.appspot.com/190079
|
|
8l: add GOOS=pchw, stop spelling out all the elf numbers.
R=r
CC=golang-dev
http://codereview.appspot.com/186144
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/186146
Committer: Russ Cox <rsc@golang.org>
|
|
* move memory code into $GOOS-specific directory.
* allow printing of static strings < 256 bytes.
(dynamic strings will bump maxstring as they are allocated.)
* use cgo2c for runtime.mal.
R=r, dho
CC=golang-dev
http://codereview.appspot.com/186143
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/186108
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/186078
|
|
in various already expensive routines.
helps keep cpu utilization up when GOMAXPROCS > 1,
but not a full solution.
http://groups.google.com/group/golang-nuts/t/7a9535c4136d3e2
R=r
CC=golang-dev
http://codereview.appspot.com/184043
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/183138
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/176066
Committer: Russ Cox <rsc@golang.org>
|
|
(Thanks to ken and rsc for pointing this out)
rsc:
ken pointed out that there's a race in the new
one-lock-per-channel code. the issue is that
if one goroutine has gone to sleep doing
select {
case <-c1:
case <-c2:
}
and then two more goroutines try to send
on c1 and c2 simultaneously, the way that
the code makes sure only one wins is the
selgen field manipulation in dequeue:
// if sgp is stale, ignore it
if(sgp->selgen != sgp->g->selgen) {
//prints("INVALID PSEUDOG POINTER\n");
freesg(c, sgp);
goto loop;
}
// invalidate any others
sgp->g->selgen++;
but because the global lock is gone both
goroutines will be fiddling with sgp->g->selgen
at the same time.
This results in a 7% slowdown in the single threaded case for a
ping-pong microbenchmark.
Since the cas predominantly succeeds, adding a simple check first
didn't make any difference.
R=rsc
CC=golang-dev
http://codereview.appspot.com/180068
|
|
them in the stream.
R=rsc
http://codereview.appspot.com/174052
|
|
most signals, so that ordinary programs
can be killed, for example.
Fixes issue 434.
R=dsymonds1
CC=golang-dev, hoisie
http://codereview.appspot.com/180064
|
|
Fixes issue 71.
R=rsc, r
http://codereview.appspot.com/162056
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 401.
R=ken2
http://codereview.appspot.com/180053
|
|
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
4th set of files.
R=rsc
CC=golang-dev
http://codereview.appspot.com/180049
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/178046
|
|
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>
|
|
registers.
R=rsc
http://codereview.appspot.com/166049
|
|
then enable stack check.
R=r
http://codereview.appspot.com/165100
|
|
FreeBSD was passing stk as the new thread's stack base, while
stk is the top of the stack in go. The added check should cause
a trap if this ever comes up in any new ports, or regresses
in current ones.
R=rsc
CC=golang-dev
http://codereview.appspot.com/167055
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/165097
Committer: Russ Cox <rsc@golang.org>
|
|
cuts working size for hello world from 6 MB to 1.2 MB.
still some work to be done, but diminishing returns.
R=r
http://codereview.appspot.com/165080
|
|
to provide functionality previously hacked in to
reflect and gob.
R=r
http://codereview.appspot.com/165076
|
|
the one-item case could be generalized easily with no cost. worth considering.
R=rsc
CC=golang-dev, cw
http://codereview.appspot.com/167044
|
|
* broken by reflect, gob
TBR=r
http://codereview.appspot.com/166077
|
|
* inform garbage collector about memory with no pointers in it
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 (this CL)
R=r
http://codereview.appspot.com/165064
|
|
Makes the code look cleaner, even if it's a little harder to figure
out from the sort invariants.
R=rsc
CC=golang-dev
http://codereview.appspot.com/165061
|
|
On a microbenchmark that ping-pongs on lots of channels, this makes
the multithreaded case about 20% faster and the uniprocessor case
about 1% slower. (Due to cache effects, I expect.)
R=rsc, agl
CC=golang-dev
http://codereview.appspot.com/166043
|
|
Fixes issue 176.
R=r
http://codereview.appspot.com/166044
|
|
* throw away dead code
* add mlookup counter
* add malloc counter
* set up for blocks with no pointers
Fixes issue 367.
R=r
http://codereview.appspot.com/165050
|
|
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=rsc, dho
http://codereview.appspot.com/157116
Committer: Russ Cox <rsc@golang.org>
|
|
we can't use them interchangably.
R=rsc, wjosephson
CC=golang-dev
http://codereview.appspot.com/156113
Committer: Russ Cox <rsc@golang.org>
|
|
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
|