Closed
Bug 1235636
Opened 9 years ago
Closed 9 years ago
PCToLineNumber incorrectly resets column number
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla46
Tracking | Status | |
---|---|---|
firefox46 | --- | fixed |
People
(Reporter: tromey, Assigned: tromey)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
fitzgen
:
review+
|
Details | Diff | Splinter Review |
I found this while working on bug 1013219 and I'm breaking it out
into a separate bug for clarity.
The bug is that PCToLineNumber conditionally updates the line number,
but unconditionally clears the column number. This means that in some
cases the column number is incorrect.
Assignee | ||
Comment 1•9 years ago
|
||
The patch.
Nick r+'d the basic idea in the other bug but I will re-request review
since I've updated some tests.
Assignee | ||
Comment 2•9 years ago
|
||
Found one more regression from this.
Attachment #8702683 -
Attachment is obsolete: true
Assignee | ||
Comment 3•9 years ago
|
||
Comment on attachment 8703016 [details] [diff] [review]
rewrite PCToLineNumber
Here's the PCToLineNumber patch separated out.
It cleans up a few mysterious column number choices in the tests.
You've already ok'd the original patch; not sure if you are able
to review the test fixes as well?
Attachment #8703016 -
Flags: review?(nfitzgerald)
Assignee | ||
Comment 4•9 years ago
|
||
Updated•9 years ago
|
Attachment #8703016 -
Flags: review?(nfitzgerald) → review+
Comment 6•9 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in
before you can comment on or make changes to this bug.
Description
•