Closed Bug 428128 Opened 16 years ago Closed 16 years ago

The demise of JSLL_ functions

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
major

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: crowderbt, Assigned: crowderbt)

Details

Attachments

(1 file, 1 obsolete file)

more removal
6.29 KB, patch
brendan
: review+
mtschrep
: approval1.9+
Details | Diff | Splinter Review
Attached patch removes ancient macros/functions (obsolete) — — Splinter Review 
Testing a browser build next; then will ask for review.
Attached patch more removal — — Splinter Review 
This builds.
Assignee: general → crowder
Attachment #314732 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #314835 - Flags: review?(brendan)
No longer depends on: 427472
Comment on attachment 314835 [details] [diff] [review]
more removal

Thanks, r=me.

/be
Attachment #314835 - Flags: review?(brendan) → review+
Comment on attachment 314835 [details] [diff] [review]
more removal

A trivial fix, proven correct by compiler-happiness.
Attachment #314835 - Flags: approval1.9?
Attachment #314835 - Flags: approval1.9? → approval1.9+
Checking in js/jsd/jsd_step.c;
/cvsroot/mozilla/js/jsd/jsd_step.c,v  <--  jsd_step.c
new revision: 3.17; previous revision: 3.16
done
Checking in js/src/jslong.c;
/cvsroot/mozilla/js/src/jslong.c,v  <--  jslong.c
new revision: 3.11; previous revision: 3.10
done
Checking in js/src/jslong.h;
/cvsroot/mozilla/js/src/jslong.h,v  <--  jslong.h
new revision: 3.13; previous revision: 3.12
done
Checking in js/src/prmjtime.c;
/cvsroot/mozilla/js/src/prmjtime.c,v  <--  prmjtime.c
new revision: 3.68; previous revision: 3.67
done
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
Flags: in-litmus-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: