Skip to content

Commit

Permalink
text/scanner: return RawString token rather than String for raw strin…
Browse files Browse the repository at this point in the history
…g literals

Fixes #23675

Change-Id: I78e13d1ca90400e4dd48674b93bb6e2e30718d97
GitHub-Last-Rev: f2b3a59
GitHub-Pull-Request: #25287
Reviewed-on: https://go-review.googlesource.com/112037
Reviewed-by: Robert Griesemer <gri@golang.org>
Run-TryBot: Robert Griesemer <gri@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
  • Loading branch information
SilverRainZ authored and griesemer committed May 8, 2018
1 parent 35ea624 commit c8915a0
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion src/text/scanner/scanner.go
Original file line number Diff line number Diff line change
Expand Up @@ -621,7 +621,7 @@ redo:
case '`':
if s.Mode&ScanRawStrings != 0 {
s.scanRawString()
tok = String
tok = RawString
}
ch = s.next()
default:
Expand Down
16 changes: 8 additions & 8 deletions src/text/scanner/scanner_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -209,10 +209,10 @@ var tokenList = []token{
{String, `"` + f100 + `"`},

{Comment, "// raw strings"},
{String, "``"},
{String, "`\\`"},
{String, "`" + "\n\n/* foobar */\n\n" + "`"},
{String, "`" + f100 + "`"},
{RawString, "``"},
{RawString, "`\\`"},
{RawString, "`" + "\n\n/* foobar */\n\n" + "`"},
{RawString, "`" + f100 + "`"},

{Comment, "// individual characters"},
// NUL character is not allowed
Expand Down Expand Up @@ -463,9 +463,9 @@ func TestError(t *testing.T) {
testError(t, `"ab`+"\x80", "<input>:1:4", "illegal UTF-8 encoding", String)
testError(t, `"abc`+"\xff", "<input>:1:5", "illegal UTF-8 encoding", String)

testError(t, "`a"+"\x00", "<input>:1:3", "illegal character NUL", String)
testError(t, "`ab"+"\x80", "<input>:1:4", "illegal UTF-8 encoding", String)
testError(t, "`abc"+"\xff", "<input>:1:5", "illegal UTF-8 encoding", String)
testError(t, "`a"+"\x00", "<input>:1:3", "illegal character NUL", RawString)
testError(t, "`ab"+"\x80", "<input>:1:4", "illegal UTF-8 encoding", RawString)
testError(t, "`abc"+"\xff", "<input>:1:5", "illegal UTF-8 encoding", RawString)

testError(t, `'\"'`, "<input>:1:3", "illegal char escape", Char)
testError(t, `"\'"`, "<input>:1:3", "illegal char escape", String)
Expand All @@ -480,7 +480,7 @@ func TestError(t *testing.T) {
testError(t, `'`+"\n", "<input>:1:2", "literal not terminated", Char)
testError(t, `"abc`, "<input>:1:5", "literal not terminated", String)
testError(t, `"abc`+"\n", "<input>:1:5", "literal not terminated", String)
testError(t, "`abc\n", "<input>:2:1", "literal not terminated", String)
testError(t, "`abc\n", "<input>:2:1", "literal not terminated", RawString)
testError(t, `/*/`, "<input>:1:4", "comment not terminated", EOF)
}

Expand Down

0 comments on commit c8915a0

Please sign in to comment.