Skip to content

Commit

Permalink
cmd/compile: ignore carriage return in magic comments
Browse files Browse the repository at this point in the history
Update golang#11771.

Change-Id: I3bb3262619765d3ca79652817e17e8f260f41907
Reviewed-on: https://go-review.googlesource.com/12379
Reviewed-by: Brad Fitzpatrick <[email protected]>
  • Loading branch information
ianlancetaylor committed Jul 18, 2015
1 parent aadd84e commit dabb268
Show file tree
Hide file tree
Showing 2 changed files with 66 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/cmd/compile/internal/gc/lex.go
Original file line number Diff line number Diff line change
Expand Up @@ -1576,7 +1576,7 @@ func getlinepragma() int {
}
cp = nil

text := lexbuf.String()
text := strings.TrimSuffix(lexbuf.String(), "\r")

if strings.HasPrefix(text, "go:cgo_") {
pragcgo(text)
Expand Down Expand Up @@ -1666,7 +1666,7 @@ func getlinepragma() int {
if linep == 0 {
return c
}
text := lexbuf.String()
text := strings.TrimSuffix(lexbuf.String(), "\r")
n := 0
for _, c := range text[linep:] {
if c < '0' || c > '9' {
Expand Down
64 changes: 64 additions & 0 deletions test/fixedbugs/issue11771.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,64 @@
// +build !nacl
// run

// Copyright 2015 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.

// Issue 11771: Magic comments should ignore carriage returns.

package main

import (
"bytes"
"fmt"
"io/ioutil"
"log"
"os"
"os/exec"
"path/filepath"
"runtime"
)

func main() {
if runtime.Compiler != "gc" {
return
}

dir, err := ioutil.TempDir("", "go-issue11771")
if err != nil {
log.Fatalf("creating temp dir: %v\n", err)
}
defer os.RemoveAll(dir)

// The go:nowritebarrier magic comment is only permitted in
// the runtime package. So we confirm that the compilation
// fails.

var buf bytes.Buffer
fmt.Fprintln(&buf, `
package main
func main() {
}
`)
fmt.Fprintln(&buf, "//go:nowritebarrier\r")
fmt.Fprintln(&buf, `
func x() {
}
`)

if err := ioutil.WriteFile(filepath.Join(dir, "x.go"), buf.Bytes(), 0666); err != nil {
log.Fatal(err)
}

cmd := exec.Command("go", "tool", "compile", "x.go")
cmd.Dir = dir
output, err := cmd.CombinedOutput()
if err == nil {
log.Fatal("compile succeeded unexpectedly")
}
if !bytes.Contains(output, []byte("only allowed in runtime")) {
log.Fatalf("wrong error message from compiler; got:\n%s\n", output)
}
}

0 comments on commit dabb268

Please sign in to comment.