Skip to content

Remove HashMap/HashSet (and even Hashable) #3132

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

Closed
DartBot opened this issue May 18, 2012 · 3 comments
Closed

Remove HashMap/HashSet (and even Hashable) #3132

DartBot opened this issue May 18, 2012 · 3 comments
Assignees
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-not-planned Closed as we don't intend to take action on the reported issue type-enhancement A request for a change that isn't a bug

Comments

@DartBot
Copy link

DartBot commented May 18, 2012

This issue was originally filed by @tomyeh


The difference between Map ans HashMap is too subtle. Moreover, it causes unnecessary confusion to Java programmers.

It is better to remove HashMap/HashSet to make the API simpler and more concise.

@iposva-google
Copy link
Contributor

Added Area-Library, Triaged labels.

@lrhn
Copy link
Member

lrhn commented Jan 11, 2013

I think we should have a plain Map interface, and implementation classes like HashMap, OrderedHashMap, and eventually also other map implementations like TreeMap or SplayMap.

Map has constructors that forward to OrderedHashMap only for convenience. They correspond to Map literals, so if you don't care what you use, just use new Map and it'll work.

@lrhn
Copy link
Member

lrhn commented Apr 22, 2013

We are sticking with the factory constructors on Map.
They currently return a HashMap. We still consider changing that to a LinkedHashMap, so "{}" and "new Map()" have the same type.
If you care about the type of Map you use, you can use that directly, and if you don't care, just use "new Map()".may


Set owner to @lrhn.
Removed Type-Defect label.
Added Type-Enhancement, NotPlanned labels.

@DartBot DartBot added Type-Enhancement area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-not-planned Closed as we don't intend to take action on the reported issue labels Apr 22, 2013
@kevmoo kevmoo added type-enhancement A request for a change that isn't a bug and removed type-enhancement labels Mar 1, 2016
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-not-planned Closed as we don't intend to take action on the reported issue type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

4 participants