Status

RESOLVED WONTFIX
9 years ago
5 years ago

People

(Reporter: graydon, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

9 years ago
The containers in nanojit/Containers.{h,cpp} have a number of issues:

  - The TreeMap should really have a balance method, even if it's delayed.
  - The HashMap limits itself to int32_t and uint32_t in places it should be using uintptr_t and intptr_t or size_t.
  - The BitSet uses int in places it should prefer size_t.
  - The Seq constructor takes a copy and makes a copy rather than taking a const ref.
  - All the associative map accessors take and return copies rather than refs.
  - The associative maps don't take hashing or comparator parameters.
  - The TreeMap should probably have an iterator as well as the HashMap.

Fix these.
Assignee: graydon → nobody
Component: JavaScript Engine → Nanojit
QA Contact: general → nanojit
(Reporter)

Comment 1

7 years ago
Long obsolete.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

5 years ago
Component: Nanojit → Nanojit
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.