| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
??? CL 105030043 / 6146799f32ed
nacltest.bash, misc/nacl/README: update NaCl docs.
LGTM=rsc
R=dave, rsc
CC=golang-codereviews
https://codereview.appspot.com/105030043
???
LGTM=r
R=golang-codereviews, r
CC=bradfitz, golang-codereviews, iant
https://codereview.appspot.com/105020044
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
name in C code
??? CL 103080043 / 5e058e21b67d
cmd/cgo: for typedef of untagged struct, use typedef name in C code
Fixes issue 8148.
LGTM=cookieo9, rsc
R=rsc, cookieo9
CC=golang-codereviews
https://codereview.appspot.com/103080043
???
LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews, r
https://codereview.appspot.com/103900046
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
??? CL 102080043 / 256d975c53cb
cmd/cgo: use same Go type for typedef to anonymous struct
If we see a typedef to an anonymous struct more than once,
presumably in two different Go files that import "C", use the
same Go type name.
Fixes issue 8133.
LGTM=rsc
R=rsc
CC=golang-codereviews
https://codereview.appspot.com/102080043
???
TBR=adg
CC=golang-codereviews
https://codereview.appspot.com/102100043
|
|
|
|
|
|
|
|
|
| |
There's no cmd/link in this branch.
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/102050043
|
|
|
|
|
|
|
| |
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/102040047
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This program has barely been touched since it was first committed,
and in its current state it opens a code execution vector similar
to the one that was recently fixed in go.tools/playground/socket.
Rather than try to make it secure, remove it.
LGTM=minux, rsc
R=rsc, minux
CC=golang-codereviews
https://codereview.appspot.com/102030047
|
|
|
|
|
|
|
| |
LGTM=bradfitz
R=jasonhall, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/91700047
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For incomplete struct S, C.T and C.struct_S were interchangeable in Go 1.2
and earlier, because all incomplete types were interchangeable
(even C.struct_S1 and C.struct_S2).
CL 76450043, which fixed issue 7409, made different incomplete types
different from Go's point of view, so that they were no longer completely
interchangeable.
However, imprecision about C.T and C.struct_S - really the same
underlying C type - is the one behavior enabled by the bug that
is most likely to be depended on by existing cgo code.
Explicitly allow it, to keep that code working.
Fixes issue 7786.
LGTM=iant, r
R=golang-codereviews, iant, r
CC=golang-codereviews
https://codereview.appspot.com/98580046
|
|
|
|
|
|
|
| |
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/99440046
|
|
|
|
|
|
|
|
|
| |
Fixes issue 8032.
LGTM=rsc
R=rsc
CC=golang-codereviews
https://codereview.appspot.com/95580043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add nacl.bash, the NaCl version of all.bash.
It's a separate script because it builds a variant of package syscall
with a large zip file embedded in it, containing all the input files
needed for tests.
Disable various tests new since the last round, mostly the ones using os/exec.
Fixes issue 7945.
LGTM=dave
R=golang-codereviews, remyoudompheng, dave, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/100590044
|
|
|
|
|
|
|
|
|
| |
Fixes issue 7835.
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/94670044
|
|
|
|
|
|
|
|
|
| |
Fixes issue 7406.
LGTM=r
R=golang-codereviews, r
CC=golang-codereviews
https://codereview.appspot.com/97440043
|
|
|
|
|
|
|
|
|
| |
Fixes issue 7560.
LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews
https://codereview.appspot.com/96300045
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If you write:
var x = 3
then the compiler arranges for x to be initialized in the linker
with an actual 3 from the data segment, rather than putting
x in the bss and emitting init-time "x = 3" assignment code.
If you write:
var y = x
var x = 3
then the compiler is clever and treats this the same as if
the code said 'y = 3': they both end up in the data segment
with no init-time assignments.
If you write
var y = x
var x int
then the compiler was treating this the same as if the
code said 'x = 0', making both x and y zero and avoiding
any init-time assignment.
This copying optimization to avoid init-time assignment of y
is incorrect if 'var x int' doesn't mean 'x = 0' but instead means
'x is initialized in C or assembly code'. The program ends up
with 'y = 0' instead of 'y = the value specified for x in that other code'.
Disable the propagation if there is no initializer for x.
This comes up in some uses of cgo, because cgo generates
Go globals that are initialized in accompanying C files.
Fixes issue 7665.
LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews
https://codereview.appspot.com/93200044
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The vi bundled with OS X has a weird bug in that if you turn off
filetype in .vimrc when it's not turned on, even a clean exit of
vi will return 1 which breaks almost everything.
While we're at it, add hint to change $GOROOT to its actual value
in .vimrc.
Fixes issue 7865.
LGTM=robert.hencke, dsymonds
R=golang-codereviews, robert.hencke, gobot, dsymonds
CC=golang-codereviews
https://codereview.appspot.com/96800046
|
|
|
|
|
|
|
|
|
|
|
|
| |
For the gc compiler the Go function Issue7695 is defined in
runtime.c, but there is no way to do that for gccgo, because
there is no way to get the correct pkgpath. The test is not
important for gccgo in any case.
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/93870044
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Cgo writes C function declarations pretending every arg is a pointer.
If the C function is deferred, it does not inhibit stack copying on split.
The stack copying code believes the C declaration, possibly misinterpreting
integers as pointers.
Probably the right fix for Go 1.3 is to make deferred C functions inhibit
stack copying.
For Go 1.4 and beyond we probably need to make cgo generate Go code
for 6g here, not C code for 6c.
Update issue 7695
LGTM=khr
R=khr
CC=golang-codereviews
https://codereview.appspot.com/83820043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
go-mode on Emacs 23 wrongly recognizes a backquote in a comment or
a string as a start of a raw string literal. Below is an example
that go-mode does not work well. This patch is to fix that issue.
// `
var x = 1
// `
LGTM=dominik.honnef
R=golang-codereviews, dominik.honnef, adonovan
CC=golang-codereviews
https://codereview.appspot.com/84900043
Committer: Alan Donovan <adonovan@google.com>
|
|
|
|
|
|
|
|
|
|
| |
People (like me!) will still try to run misc/benchcmp
and wonder where it went. Tell them.
LGTM=bradfitz
R=golang-codereviews, bradfitz, dave
CC=adg, golang-codereviews, r
https://codereview.appspot.com/82710043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Name of built-in function is not reserved word in Go, and you can
use it as variable name. "new" is often used as local variable, for
instance.
This patch is to apply font-lock-builtin-face only when built-in
function name is followed by '(', so that it doesn't highlight
non-function variable that happen to have the same name as built-in
function.
LGTM=dominik.honnef
R=golang-codereviews, dominik.honnef, adonovan
CC=golang-codereviews
https://codereview.appspot.com/79260043
Committer: Alan Donovan <adonovan@google.com>
|
|
|
|
|
|
|
|
|
|
|
| |
cgo represents all 0-sized and unsized types internally as [0]byte. This means that pointers to incomplete types would be interchangable, even if given a name by typedef.
Fixes issue 7409.
LGTM=iant
R=golang-codereviews, bradfitz, iant
CC=golang-codereviews
https://codereview.appspot.com/76450043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Go-mode in Emacs 23 does not recognize a backslash followed
by a backquote as end of raw string literal, as it does not
support syntax-propertize-function which Go-mode uses to
remove special meaning from backslashes in ``.
This patch provides a fallback mechanism to do the same thing
using font-lock-syntactic-keywords, which is supported by
Emacs 23.
LGTM=dominik.honnef
R=golang-codereviews, dominik.honnef
CC=adonovan, golang-codereviews
https://codereview.appspot.com/78730048
Committer: Alan Donovan <adonovan@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch includes fixes pointed out in CL 52140043, which was
originally written by john.gosset.
LGTM=minux.ma
R=golang-codereviews, minux.ma
CC=golang-codereviews
https://codereview.appspot.com/80320043
Committer: Shenghou Ma <minux.ma@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If someone configures a 'textwidth' in go files, vim will by default insert
newlines into long lines as you type, which breaks syntax and doesn't really
make sense for go code. This fixes the default.
LGTM=dsymonds
R=golang-codereviews, gobot, dsymonds
CC=golang-codereviews
https://codereview.appspot.com/76890046
Committer: David Symonds <dsymonds@golang.org>
|
|
|
|
|
|
|
| |
LGTM=josharian, dan.kortschak
R=golang-codereviews, josharian, dan.kortschak
CC=golang-codereviews
https://codereview.appspot.com/75080043
|
|
|
|
|
|
|
|
|
|
|
| |
"go doc" has been removed in CL 17799.
LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews
https://codereview.appspot.com/77420045
Committer: Ian Lance Taylor <iant@golang.org>
|
|
|
|
|
|
|
|
|
|
|
| |
"env" is a valid go command. This patch is to make bash to autocomplete it.
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/74660045
Committer: Brad Fitzpatrick <bradfitz@golang.org>
|
|
|
|
|
|
|
|
|
|
|
| |
We were fontifying the wrong part of method declarations
LGTM=adonovan
R=adonovan
CC=golang-codereviews
https://codereview.appspot.com/66120043
Committer: Alan Donovan <adonovan@google.com>
|
|
|
|
|
|
|
|
|
|
| |
This broke when we renamed the tool, and I missed this fix when I
fixed darwin last week.
LGTM=minux.ma, bradfitz
R=golang-codereviews, minux.ma, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/70670043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We're about to commit some wide-sweeping changes to the go-tour and I
would rather not include them in Go 1.2.1, which is due in the next
week or so.
Also fix the makerelease tool; it has been broken since it was
renamed from bindist.
LGTM=campoy
R=campoy
CC=golang-codereviews
https://codereview.appspot.com/68780043
|
|
|
|
|
|
|
|
|
| |
Fix build for 10.6 Darwin builders and OpenBSD builers.
LGTM=jsing
R=golang-codereviews, dave, jsing
CC=golang-codereviews
https://codereview.appspot.com/67710043
|
|
|
|
|
|
|
|
|
|
|
| |
Fix build for Dragonfly BSD.
Fixes issue 7318.
Fixes issue 7367.
LGTM=jsing, iant
R=jsing, iant, mikioh.mikioh
CC=golang-codereviews
https://codereview.appspot.com/64340043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
c-mode classically uses ff-find-other-file to toggle between headers
and implementation. For Go it seemingly makes sense to jump between
implementation and test.
While there's no enforced mapping of file names for tests, the mapping
in this CL seems to be very common at least throughout the standard
library, and ff-find-other-file fails gracefully when the mapping
doesn't apply.
LGTM=adonovan
R=adonovan
CC=golang-codereviews
https://codereview.appspot.com/65750044
Committer: Alan Donovan <adonovan@google.com>
|
|
|
|
|
|
|
|
|
| |
LGTM=adonovan
R=adonovan
CC=golang-codereviews
https://codereview.appspot.com/62600043
Committer: Alan Donovan <adonovan@google.com>
|
|
|
|
|
|
|
|
|
|
| |
Better sampling of objects that are close in size to sampling rate.
See the comment for details.
LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://codereview.appspot.com/43830043
|
|
|
|
|
|
|
| |
LGTM=bradfitz
R=golang-codereviews, bradfitz, alex.brainman
CC=golang-codereviews
https://codereview.appspot.com/61260044
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes issue 7016.
LGTM=r
R=r
CC=adg, bradfitz, dave, golang-codereviews
https://codereview.appspot.com/60100043
Committer: Rob Pike <r@golang.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In external link mode the linker explicitly adds the string
constant "runtime/cgo". It adds the string constant using the
same symbol name as the compiler, but a different format. The
compiler assumes that the string data immediately follows the
string header, but the linker puts the two in different
sections. The result is bad string data when the compiler
sees "runtime/cgo" used as a string constant.
The compiler assumption is in datastring in [568]g/gobj.c.
The linker layout is in addstrdata in ld/data.c. The compiler
assumption is valid for string literals. The linker is not
creating a string literal, so its assumption is also valid.
There are a few ways to avoid this problem. This patch fixes
it by only doing the fake import of runtime/cgo if necessary,
and by only creating the string symbol if necessary.
Fixes issue 7234.
LGTM=dvyukov
R=golang-codereviews, dvyukov, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/58410043
|
|
|
|
|
|
|
|
|
| |
Now it should always fail on ARM.
(The fix is on its way too.)
R=iant, r, dave
CC=golang-codereviews
https://codereview.appspot.com/55140043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
NPTL uses SIGRTMIN (signal 32) to effect thread cancellation.
Go's runtime replaces NPTL's signal handler with its own, and
ends up aborting if a C library that ends up calling
pthread_cancel is used.
This patch prevents runtime from replacing NPTL's handler.
Fixes issue 6997.
R=golang-codereviews, iant, dvyukov
CC=golang-codereviews
https://codereview.appspot.com/47540043
Committer: Ian Lance Taylor <iant@golang.org>
|
|
|
|
|
|
|
|
|
|
|
| |
Require at least one space after "type" and do not fontify closing
parenthesis of type switch as a type.
R=adonovan
CC=golang-codereviews
https://codereview.appspot.com/37720050
Committer: Alan Donovan <adonovan@google.com>
|
|
|
|
|
|
|
|
| |
R=adonovan
CC=golang-codereviews
https://codereview.appspot.com/23290044
Committer: Alan Donovan <adonovan@google.com>
|
|
|
|
|
|
|
|
| |
R=golang-codereviews, gobot, dsymonds
CC=golang-codereviews
https://codereview.appspot.com/44960043
Committer: David Symonds <dsymonds@golang.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since version 6.5, npp change the Function List syntax for User Defined Languages.
We need use userDefinedLangName syntax in association tag in Notepad++ 6.5.
Fix issue 6735.
R=golang-dev, rsc
CC=golang-dev
https://codereview.appspot.com/22770043
Committer: Brad Fitzpatrick <bradfitz@golang.org>
|
|
|
|
|
|
|
|
| |
Fixes issue 6841.
R=golang-dev, dsymonds
CC=golang-dev
https://codereview.appspot.com/43010044
|
|
|
|
|
|
|
|
| |
The ambiguity has gone on too long.
R=golang-dev, minux.ma, r, dsymonds
CC=golang-dev
https://codereview.appspot.com/39920043
|
|
|
|
|
|
| |
R=golang-dev, rsc
CC=golang-dev
https://codereview.appspot.com/39780043
|
|
|
|
|
|
|
|
| |
Fixes issue 6833
R=minux.ma, iant
CC=golang-dev
https://codereview.appspot.com/35790045
|
|
|
|
|
|
|
|
| |
Output from gofmt is a list of errors, so they should appear in the error list.
R=adg
CC=golang-dev
https://codereview.appspot.com/33760043
|