Closed
Bug 307278
Opened 20 years ago
Closed 20 years ago
Remove the JS shell
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mrbkap, Assigned: mrbkap)
Details
Attachments
(1 file)
|
31.53 KB,
patch
|
brendan
:
review+
|
Details | Diff | Splinter Review |
This folder (and the files inside of it) haven't been touched since 1999. I took
a look inside and found buffer overruns and other fun stuff inside (I haven't
tried to compile it or verify that it even tries to work). In its defense, I
didn't see nearly as many overt bugs as in jsfile.c, but that isn't a very hard
metric to meet.
I'll attach a diff -N of the removal in a second.
| Assignee | ||
Comment 1•20 years ago
|
||
Attachment #195075 -
Flags: review?(brendan)
Comment 2•20 years ago
|
||
Comment on attachment 195075 [details] [diff] [review]
diff -N of the removal
Buh-bye! CVS never forgets, but we can.
Onward to renaming:
js/src/xpconnect => js/xpcom
js/src/liveconnect => js/java
js/src/perlconnect => js/perl
I believe I can help do the CVS moves.
/be
Attachment #195075 -
Flags: review?(brendan) → review+
| Assignee | ||
Comment 3•20 years ago
|
||
One bug fix per bug is my current mantra. Marking this as FIXED, the JS shell
implementation is now residing in the Attic.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Flags: testcase-
You need to log in
before you can comment on or make changes to this bug.
Description
•