]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/cgo: don't define intgo in export prologue
authorIan Lance Taylor <iant@golang.org>
Thu, 14 Dec 2017 01:29:28 +0000 (17:29 -0800)
committerIan Lance Taylor <iant@golang.org>
Thu, 14 Dec 2017 03:38:19 +0000 (03:38 +0000)
The export prologue goes into the _cgo_export.h file, where it may be
be #include'd by a .swig file. As SWIG defines its own type "intgo",
the definition of "intgo" in the export prologue could conflict.
Since we don't need to define "intgo" in the _cgo_export.h file, don't.

Defining "intgo" in _cgo_export.h was new for this release, so this
should not break any existing code.

No test case as I can't quite bring myself to write a test that
combines SWIG and cgo.

Change-Id: I8073e8300a1860cecd5994b9ad07dd35a4298c89
Reviewed-on: https://go-review.googlesource.com/83936
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/cmd/cgo/out.go

index 8834c3db5a177b76fa986c0ed5019e352cfd96f1..44a00d8154f8caa1ff0361c4cf00dcfd74604aea 100644 (file)
@@ -1652,9 +1652,7 @@ const builtinExportProlog = `
 #ifndef GO_CGO_EXPORT_PROLOGUE_H
 #define GO_CGO_EXPORT_PROLOGUE_H
 
-typedef ptrdiff_t intgo;
-
-typedef struct { const char *p; intgo n; } _GoString_;
+typedef struct { const char *p; ptrdiff_t n; } _GoString_;
 
 #endif
 `