webrev: sdiff view indentation does not account for width line number

XMLWordPrintable

    • Type: Bug
    • Resolution: Fixed
    • Priority: P3
    • None
    • Affects Version/s: None
    • Component/s: cli
    • None

      The sdiff view of webrev seems to not account for the width of the line numbers. With every additional digit, the code is shifted an additional character to the right.

      This can be seen in the attached sdiff file (from the JDK-8183372, fwiw), at the transition from lines 9->10 and 99->100.

      When reviewing code, this looks like a phantom indentation problem in the code change.

            Assignee:
            Erik Helin
            Reporter:
            Brent Christian
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: