]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/cgo: add column number to line directives
authorKir Kolyshkin <kolyshkin@gmail.com>
Mon, 30 Jul 2018 14:05:49 +0000 (17:05 +0300)
committerIan Lance Taylor <iant@golang.org>
Tue, 31 Jul 2018 20:20:47 +0000 (20:20 +0000)
Due to a new specification of //line: directives, missing
column info is now treated as column 0, aka "unknown column"
(see https://github.com/golang/go/issues/24183 for details).

As cgo does not add column number to generated //line: directive,
resulting files parsed do not have column info.

Fix by adding column of 1 to generated line directives.

Fixes #26692

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

index 07874974eeb63ced6b907ee7b7aaebdb944237bc..89598c96e8300f911f064bc5fde5225965ef3342 100644 (file)
@@ -522,7 +522,7 @@ func (p *Package) writeOutput(f *File, srcfile string) {
 
        // Write Go output: Go input with rewrites of C.xxx to _C_xxx.
        fmt.Fprintf(fgo1, "// Code generated by cmd/cgo; DO NOT EDIT.\n\n")
-       fmt.Fprintf(fgo1, "//line %s:1\n", srcfile)
+       fmt.Fprintf(fgo1, "//line %s:1:1\n", srcfile)
        fgo1.Write(f.Edit.Bytes())
 
        // While we process the vars and funcs, also write gcc output.