My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 3499: Go syntax highlighting misparses backslashes in raw string literals (e.g., `\`)
3 people starred this issue and may be notified of changes. Back to list
Status:  Released
Owner:  ----
Closed:  Oct 7


Sign in to add a comment
 
Reported by mdemp...@google.com, Jul 28, 2015
Affected Version: 2.11.2-1071-g6687cd4

What steps will reproduce the problem?
1. https://go-review.googlesource.com/#/c/12779/1/src/cmd/dist/build.go@88

What is the expected output? What do you see instead?
`\` is a one-character raw string literal, as backslash sequences are not interpreted within the backquotes.
However, instead Gerrit seems to think the backslash is escaping the backquote, and the raw string literal continues until line 833.


Please provide any additional information below.
http://golang.org/ref/spec#String_literals
Jul 28, 2015
#1 dborowitz@google.com
Gerrit consumes the syntax highlighters directly from CodeMirror, so you may have better luck reporting this at https://github.com/codemirror/codemirror/issues
Aug 3, 2015
#3 mdemp...@google.com
Fixed upstream in CodeMirror.
Aug 3, 2015
Project Member #4 david.pu...@sonymobile.com
If they continue the same release cadence as the past few, a new one will be out in a few weeks.

Are you OK with waiting for the next release, or do you want it now?  If so I can spin a new custom build.

Aug 4, 2015
#5 mdemp...@google.com
No rush. Waiting a few weeks is fine by me.
Aug 17, 2015
#6 tw201...@gmail.com
BTW: duplicate  issue 3164 .
Aug 17, 2015
Project Member #7 edwin.ke...@gmail.com
 Issue 3164  has been merged into this issue.
Sep 13, 2015
Project Member #8 david.pu...@sonymobile.com
https://gerrit-review.googlesource.com/#/c/70880/
Status: ChangeUnderReview
Oct 7, 2015
Project Member #9 ekem...@google.com
(No comment was entered for this change.)
Status: Submitted
Labels: FixedIn-2.12
Dec 21, 2015
Project Member #10 david.pu...@sonymobile.com
(No comment was entered for this change.)
Status: Released
Sign in to add a comment

Powered by Google Project Hosting