Issue 250: syntax highlighting fails when gerrit breaks long lines
Status:  Released
Owner: ----
Closed:  Oct 2012

Blocked on:
issue 251
issue 363
Reported by code-rev...@gtempaccount.com, Sep 24, 2009
Reported by Shawn Pearce <sop@google.com> on Mon Jul 20 15:00:15 PDT 2009
Source: JIRA GERRIT-251
Affected Version: 2.0.17

When Gerrit finds a source code line longer than 100 characters it
automatically breaks the line to make it easier to read files in a side-by-
side view on a laptop.  However we break before the source code pretty
formatter is applied, so the pretty formatter isn't able to render quoted
strings correctly.  For example:

  http://review.source.android.com/#patch,sidebyside,10721,1,src/main/java/com/google/gerrit/git/MergeOp.java

Look at new version, line 780.  The line should be styled as a quoted string,
like line 769 is.  The line break we insert causes the formatter to be unable
to match the string literal here.
Sep 24, 2009
#1 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Wed Aug 19 10:34:04 PDT 2009

Also related to GERRIT-252.
Sep 24, 2009
#2 sop+code@google.com
(No comment was entered for this change.)
Status: Accepted
Owner: ---
Blockedon: 251
Dec 18, 2009
#3 sop@google.com
(No comment was entered for this change.)
Blockedon: 363
Jan 30, 2010
#4 sop@google.com
Fixed by change I155bb7abc560f01a3597b3be678a76a5aa7f9e68
Status: Fixed
Labels: FixedIn-2.1.2
Oct 21, 2012
#5 sop@google.com
(No comment was entered for this change.)
Status: Released
Blockedon: -gerrit:251 -gerrit:363 gerrit:251 gerrit:363
Blocking: -gerrit:251