Closed Bug 500128 Opened 11 years ago Closed 11 years ago
Long js files (longer than 65535 lines long ?) have incorrect line numbers
626 bytes, text/html
89.36 KB, application/x-bzip2
3.85 KB, patch
|Details | Diff | Splinter Review|
Originally reported at Firebug http://code.google.com/p/fbug/issues/detail?id=1900 The test case is 70000 lines of comments and a function that has an error at line 70003. The error is reported at line 4467. This is just one symptom. Breakpoints etc don't work. User thinks the lines wrap. This problem is going to be suddenly very big. Both dojo and ExtJS have now gone over 60kloc and devs have discovered that loading one be JS file is much faster than 30 smaller files. The test code should appear at http://getfirebug.com/tests/issues/1900 whenever the server thinks its time.
This is fixed in 3.5, by the patch for bug 490818. /be
Status: NEW → RESOLVED
Closed: 11 years ago
Depends on: 490818
Resolution: --- → FIXED
My tests on Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1) Gecko/20090616 Firefox/3.5 Still gives line 4467 for its error message. The function that is created ('test') has a baseLineNumber of 1 and lineExtent of 4471. The correct answer would be 70002 and 3 or so. jjb
This isn't fixed.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Not fixed in Fx 3.5.2 and not fixed in Fx 3.6a1
There's a whole bunch of casts to uint16 in jsscan.cpp and jsscan.h that probably need to be updated. Can someone take a look at the attached patch?
Oh good grief! Ed, thanks for finding the bug. The patch can be simplified not to cast at all, though. The casts originally were to silence warnings and signal intent to chop to uint16, in the days of mythical <64Kline JS sources. /be
My fault, I'll get this revised version of Ed's patch, r=me, checked into the tracemonkey repo right away. Thanks again, /be
Fixed in tm: http://hg.mozilla.org/tracemonkey/rev/deb19a62f1f9 JJB, anyone: please nominate for branches as appropriate. Thanks, /be
OS: Windows XP → All
Hardware: x86 → All
Whiteboard: [firebug-p1] → [firebug-p1] fixed-in-tracemonkey
Whiteboard: [firebug-p1] fixed-in-tracemonkey → [firebug-p1] fixed-in-tracemonkey [waiting on tracemonkey merge]
Status: ASSIGNED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Whiteboard: [firebug-p1] fixed-in-tracemonkey [waiting on tracemonkey merge] → [firebug-p1] fixed-in-tracemonkey [waiting on tracemonkey merge][needs-approval-192][needs-checkin-192]
You need to log in before you can comment on or make changes to this bug.