New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename LinkedHashMap to OrderedMap #2349
Comments
This comment was originally written by jjb@google.com Better still is to get rid of the interface entirely, and simply implement the Map interface. |
Will there be an implementation of an ordered map? If so, thoughts on name? |
This comment was originally written by jjb@google.com I was going to keep the name LinkedHashMap. InsertionOrderedMap would be another possibility, though LinkedHashMap is familiar to Java programmers. Do you believe that InsertionOrderedMap would be a much better name for Javascript programmers? |
I'm at a JavaScript summit right now actually and LinkedHashMap was unfamiliar to this crowd. You're right, it's very Java. InsertionOrderedMap is more clear about "what it does", as LinkedHashMap sounds like "how it does it" |
Recategorizing as this has nothing to do with language area. Removed Area-Language label. |
cc @lrhn. |
Thoughts on this? I see LinkedHashMap is still in trunk. http://api.dartlang.org/docs/trunk/latest/dart_core/LinkedHashMap.html cc @floitschG. |
It's on my TODO list. Added Accepted label. |
This comment was originally written by @sethladd I see this doesn't have a Milestone. I think it missed M3, will it get into M4? (I ask because I want to know what kind of disclaimer to add to some docs :) |
Added this to the M4 milestone. |
The name LinkedHashMap is not clear in its intent. We suggest OrderedMap as a better interface name. We suspect this name will be easier to understand and identify for JavaScript developers browsing the APIs.
The text was updated successfully, but these errors were encountered: