Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Use a count of -1 for infinity. Ditto for Replace.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1704044
Committer: Rob Pike <r@golang.org>
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1693047
|
|
libmach: disassemble CMPPD as 6a expects
R=ken2
CC=Charlie Dorian, golang-dev
http://codereview.appspot.com/1704046
|
|
(Here, quoted strings are the official AMD names.)
The amd64 "movsxd" instruction, when invoked
with a 64-bit REX prefix, moves and sign extends
a 32-bit value from register or memory into a
64-bit register. 6.out.h spells this MOVLQSX.
6.out.h also includes MOVLQZX, the zero extending
version, which it implements as "movsxd" without
the REX prefix. Without the REX prefix it's only sign
extending 32 bits to 32 bits (i.e., not doing anything
to the bits) and then storing in a 32-bit register.
Any write to a 32-bit register zeros the top half of the
corresponding 64-bit register, giving the advertised effect.
This particular implementation of the functionality is
non-standard, because an ordinary 32-bit "mov" would
do the same thing.
Because it is non-standard, it is often mishandled or
not handled by binary translation tools like valgrind.
Switching to the standard "mov" makes the binaries
work better with those tools.
It's probably useful in 6c and 6g to have an explicit
instruction, though, so that the intent of the size
change is clear. Thus we leave the concept of MOVLQZX
and just implement it by the standard "mov" instead of
the non-standard 32-bit "movsxd".
Fixes issue 896.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1733046
|
|
With these changes, goinstall is now able to use branches
maintained with Bazaar located in Launchpad.
Project aliases such as /project and /project/series are
supported in addition to specific user or team branches
such as /~user/project/branch. Temporary branches under
the +junk special project are also supported.
As a curious side effect, since Launchpad is able to import
code from other locations, they can be indirectly
accessible too if desired.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1699050
Committer: Russ Cox <rsc@golang.org>
|
|
The Makefile and cgo now rewrite / to _ when creating the path.
The .so for gosqlite.googlecode.com/hg/sqlite is named
cgo_gosqlite.googlecode.com_hg_sqlite.so, and then 6l and 8l
both include a default rpath of $GOROOT/pkg/$GOOS_$GOARCH.
This should make it easier to move binaries from one system
to another.
Fixes issue 857.
R=iant, r
CC=golang-dev
http://codereview.appspot.com/1700048
|
|
Fixes issue 868.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1695049
|
|
Fixes issue 879.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1678048
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1729046
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1678046
|
|
fixes issue 892
R=rsc
CC=golang-dev
http://codereview.appspot.com/1745042
|
|
fixes issue 867.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1691045
|
|
fixes issue 887.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1675050
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1677049
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1720043
|
|
R=r, r2
CC=golang-dev
http://codereview.appspot.com/1707043
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1667048
|
|
Currently to install a command, you have to manually
goinstall each of the remote packages that it depends on.
This patch lets goinstall P work where P is
contains files in package main.
It does not actually build the package, but
it installs all of its dependencies and prints a message
to that effect.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1301043
Committer: Russ Cox <rsc@golang.org>
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1736042
|
|
update other code to match.
R=r
CC=golang-dev
http://codereview.appspot.com/1680044
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1732043
|
|
x.go:13: cannot use t (type T) as type Reader in assignment:
T does not implement Reader (Read method requires pointer receiver)
x.go:19: cannot use q (type Q) as type Reader in assignment:
Q does not implement Reader (missing Read method)
have read()
want Read()
x.go:22: cannot use z (type int) as type Reader in assignment:
int does not implement Reader (missing Read method)
x.go:24: too many arguments to conversion to complex: complex(1, 3)
R=ken2
CC=golang-dev
http://codereview.appspot.com/1736041
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1677047
|
|
R=rsc
TBR=rsc
CC=golang-dev
http://codereview.appspot.com/1675047
|
|
various cleanup, deleting unused code
R=ken2
CC=golang-dev
http://codereview.appspot.com/1663041
|
|
Cleans up a few other corner cases too.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1592045
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1597044
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1627042
|
|
There's only one Go object file per package now,
so there's no need to parse the metadata and merge
metadata from multiple files. Just save the original
and use it as __.PKGDEF verbatim.
R=r
CC=golang-dev
http://codereview.appspot.com/1647042
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1597043
|
|
This avoids a crash when using cgo where glibc's malloc thinks
that it can use some of the memory following the symbol table.
This fails because the symbol table is mapped read-only, which
affects the whole page.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1616042
|
|
R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/1636043
|
|
* disallow surrogate pair runes.
* diagnose impossible type assertions
* eliminate another static buffer.
* do not overflow lexbuf.
* add -u flag to disable package unsafe.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1619042
|
|
* Code for assignment, conversions now mirrors spec.
* Changed some snprint -> smprint.
* Renamed runtime functions to separate
interface conversions from type assertions:
convT2I, assertI2T, etc.
* Correct checking of \U sequences.
Fixes issue 840.
Fixes issue 830.
Fixes issue 778.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1303042
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1621041
|
|
Fixes issue 844.
R=ken2, rsc, ken3
CC=golang-dev
http://codereview.appspot.com/1593041
|
|
Fixes issue 827.
R=ken2
CC=golang-dev, rsc
http://codereview.appspot.com/1592041
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1514041
Committer: Russ Cox <rsc@golang.org>
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1442042
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1395041
Committer: Russ Cox <rsc@golang.org>
|
|
It turns out that _cgo_malloc is used, via cmalloc in
runtime/cgocall.c, which is called by code generated by out.go
for the ·_C_CString function. I can't find a call to
_cgo_free, but given _cgo_malloc we might as well keep
_cgo_free. This patch fixes it so that it should work on
amd64.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1399041
|