| Issue 251: | Multi-line comment support honors /* inside strings | |
| 3 people starred this issue and may be notified of changes. | Back to list |
Sign in to add a comment
|
Reported by Jeff Hamilton <jham@android.com> on Tue Jul 21 08:08:38 PDT 2009
Source: JIRA GERRIT-252
Affected Version: 2.0.17
Environment: FF 3.5.1 on OS X 10.5.7
I was reviewing a source file with a line similar to:
matcher.addURI("foo", "foo/bar/*",
and Gerrit thought that the following lines were inside a multi-line comment.
Sep 24, 2009
#1
code-rev...@gtempaccount.com
Sep 24, 2009
Comment by Shawn Pearce <sop@google.com> on Wed Aug 12 07:00:59 PDT 2009 Also in https://review.source.android.com/#patch,sidebyside,11110,1,src/test/java/com/google/gerrit/server/query/QueryParserTest.java starts on RHS line 78.
Sep 24, 2009
(No comment was entered for this change.)
Status:
Accepted
Owner: ---
Sep 24, 2009
(No comment was entered for this change.)
Blockedon:
250
Sep 24, 2009
(No comment was entered for this change.)
Cc:
j...@android.com
Dec 18, 2009
(No comment was entered for this change.)
Blockedon:
363
Jan 19, 2010
Issue 398 has been merged into this issue.
Jan 30, 2010
Fixed by change I155bb7abc560f01a3597b3be678a76a5aa7f9e68
Status:
Fixed
Labels: FixedIn-2.1.2
Feb 13, 2010
Issue 449 has been merged into this issue.
Mar 27, 2012
(No comment was entered for this change.)
Status:
Released
|
||||||||||
| ► Sign in to add a comment | |||||||||||