Closed Bug 486124 Opened 15 years ago Closed 15 years ago

TCF_HAS_DEFXMLNS is deadwood

Categories

(Core :: JavaScript Engine, enhancement)

Product:
Core
Component:
JavaScript Engine
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: igor, Assigned: igor)

Details

(Keywords: fixed1.9.1, Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file)

v1
4.30 KB, patch
mrbkap
: review+
Details | Diff | Splinter Review 
Currently the code only sets TCF_HAS_DEFXMLNS but it never checks for the flag. It would be nice to remove this deadwood.
Attached patch v1 — — Splinter Review 
The patch is a straightforward removal of TCF_HAS_DEFXMLNS.
Attachment #370244 - Flags: review?(mrbkap)
Nominating for 1.9.1 not to carry there unused code.
Flags: wanted1.9.1?
Attachment #370244 - Flags: review?(mrbkap) → review+
landed to TM - http://hg.mozilla.org/tracemonkey/rev/aa5ea87b5cfb
Whiteboard: fixed-in-tracemonkey
http://hg.mozilla.org/mozilla-central/rev/aa5ea87b5cfb
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Flags: wanted1.9.1? → wanted1.9.1+
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: