Opened 14 years ago
Closed 14 years ago
#3372 closed defect (fixed)
0.9: Remove _itemMap in JsonItemStore, not needed with keys as numerics.
Reported by: | Jared Jurkiewicz | Owned by: | Jared Jurkiewicz |
---|---|---|---|
Priority: | high | Milestone: | 0.9beta |
Component: | Data | Version: | 0.9 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
0.9: Remove _itemMap in JsonItemStore?, not needed with keys as numerics.
The original idea behind having the map was so that non-numrerics could be used as the ids as well in the O(1) isItem, but that wasn't how it ended up being implemented. So, realistically, the single arrayOfAllItems can be used in its place. Removing the map and slightly altering isItem. More efficient, too.
Attachments (1)
Change History (2)
Changed 14 years ago by
Attachment: | dojo.data_JsonItemStore_20070612.patch added |
---|
comment:1 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
(In [9047]) Trivial cleanup. fixes #3372