Closed
Bug 509637
Opened 15 years ago
Closed 12 years ago
NJ: container nits
Categories
(Core Graveyard :: Nanojit, defect)
Core Graveyard
Nanojit
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: graydon, Unassigned)
Details
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.
Updated•13 years ago
|
Assignee: graydon → nobody
Component: JavaScript Engine → Nanojit
QA Contact: general → nanojit
Reporter | ||
Comment 1•12 years ago
|
||
Long obsolete.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•10 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•