Issue 251: Multi-line comment support honors /* inside strings
Status:  Released
Owner: ----
Closed:  Mar 2012
Cc:

Blocked on:
issue 250
issue 363

Blocking:
issue 250
Reported by code-rev...@gtempaccount.com, Sep 24, 2009
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
Comment by Shawn Pearce <sop@google.com> on Tue Jul 21 08:12:59 PDT 2009

https://android-git.corp.google.com/g/#patch,sidebyside,7978,1,src/com/android/providers/contacts/ContactsProvider2.java

The problem starts on RHS line 327.
Sep 24, 2009
#2 code-rev...@gtempaccount.com
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
#3 sop+code@google.com
(No comment was entered for this change.)
Status: Accepted
Owner: ---
Sep 24, 2009
#4 sop+code@google.com
(No comment was entered for this change.)
Blockedon: 250
Sep 24, 2009
#5 sop+code@google.com
(No comment was entered for this change.)
Cc: j...@android.com
Dec 18, 2009
#6 sop@google.com
(No comment was entered for this change.)
Blockedon: 363
Jan 19, 2010
#7 sop@google.com
 Issue 398  has been merged into this issue.
Jan 30, 2010
#8 sop@google.com
Fixed by change I155bb7abc560f01a3597b3be678a76a5aa7f9e68
Status: Fixed
Labels: FixedIn-2.1.2
Feb 13, 2010
#9 sop@google.com
 Issue 449  has been merged into this issue.
Mar 27, 2012
#10 sop@google.com
(No comment was entered for this change.)
Status: Released