diff --git a/misc/cgo/test/empty.go b/misc/cgo/test/empty.go deleted file mode 100644 index daa7485492..0000000000 --- a/misc/cgo/test/empty.go +++ /dev/null @@ -1,18 +0,0 @@ -// Copyright 2014 The Go Authors. All rights reserved. -// Use of this source code is governed by a BSD-style -// license that can be found in the LICENSE file. - -package cgotest - -/* -#cgo CFLAGS: -Werror=unused-variable -void funcWithoutAnyParams() {} -*/ -import "C" - -// Only test whether this can be compiled, unused -// variable (e.g. empty gcc strut) could cause -// warning/error under stricter CFLAGS. -func testEmptyGccStruct() { - C.funcWithoutAnyParams() -} diff --git a/src/cmd/cgo/out.go b/src/cmd/cgo/out.go index a5fffd0568..c6c27c4dbf 100644 --- a/src/cmd/cgo/out.go +++ b/src/cmd/cgo/out.go @@ -517,7 +517,7 @@ func (p *Package) writeOutputFunc(fgcc *os.File, n *Name) { return } - ctype, offset := p.structType(n) + ctype, _ := p.structType(n) // Gcc wrapper unpacks the C argument struct // and calls the actual C function. @@ -530,9 +530,7 @@ func (p *Package) writeOutputFunc(fgcc *os.File, n *Name) { // We're trying to write a gcc struct that matches 6c/8c/5c's layout. // Use packed attribute to force no padding in this struct in case // gcc has different packing requirements. - if offset != 0 { - fmt.Fprintf(fgcc, "\t%s %v *a = v;\n", ctype, p.packedAttribute()) - } + fmt.Fprintf(fgcc, "\t%s %v *a = v;\n", ctype, p.packedAttribute()) fmt.Fprintf(fgcc, "\t") if t := n.FuncType.Result; t != nil { fmt.Fprintf(fgcc, "a->r = ")