Release a public AbstractBiMap or BiMaps.newBiMap(Map, Map) method

[GoogleCodeExporter]

I think this is one very strong class in the API and you made it package 
viewable only. So I can't for example build a BiMap backed by my own map 
implementations or some standard java impl. like LinkedHashMap for 
insertion-order iteration.

Original issue reported on code.google.com by nimrod.priell on 12 Mar 2009 at 8:19

[GoogleCodeExporter]

It's always been a future possibility, but designing and documentation for
inheritance is a painful, very time-consuming activity and is not a priority 
for 1.0.

Bear in mind that ImmutableBiMap is insertion-ordered, and that 
insertion-ordered
*mutable* bimaps can be kind of confusing because their key set and value set 
don't
iterate in the corresponding order.

Original comment by kevin...@gmail.com on 12 Mar 2009 at 6:39

[GoogleCodeExporter]

Instead of making StandardBiMap public, we could instead create a 
BiMaps.newBiMap()
method that takes backing maps as input parameters. That approach avoids 
problems
that arise when collection classes are publicly extendable.

We currently don't have that method because, until now, nobody has asked for it.

Original comment by jared.l....@gmail.com on 12 Mar 2009 at 6:43

[GoogleCodeExporter]

Actually, we used to have it, for a long time, internally, and then we noticed 
that
of the few people who used it, none of them were actually getting the benefit 
from it
they thought they were.  We decided after giving it a chance that this was not 
very
useful, and we got rid of it.

Original comment by kevin...@gmail.com on 12 Mar 2009 at 6:46

[GoogleCodeExporter]

Correction: Maps.newBiMap()

Original comment by jared.l....@gmail.com on 12 Mar 2009 at 6:48

[GoogleCodeExporter]

Original comment by kevin...@gmail.com on 12 Sep 2009 at 2:14

• Changed title: Release a public AbstractBiMap or BiMaps.newBiMap(Map, Map) method
• Added labels: post-1.0

[GoogleCodeExporter]

Issue 235 has been merged into this issue.

Original comment by kevin...@gmail.com on 12 Sep 2009 at 2:15

[GoogleCodeExporter]

Original comment by kevin...@gmail.com on 17 Sep 2009 at 5:45

• Added labels: Milestone-Post1.0
• Removed labels: post-1.0

[GoogleCodeExporter]

Original comment by kevin...@gmail.com on 17 Sep 2009 at 5:57

• Added labels: Type-Enhancement

[GoogleCodeExporter]

Issue 283 has been merged into this issue.

Original comment by kevin...@gmail.com on 30 Oct 2009 at 4:00

[GoogleCodeExporter]

Original comment by kevinb@google.com on 30 Jul 2010 at 3:53

• Removed labels: Milestone-Post1.0

[GoogleCodeExporter]

Original comment by fry@google.com on 26 Jan 2011 at 9:50

• Changed state: Accepted

[GoogleCodeExporter]

Original comment by kevinb@google.com on 13 Jul 2011 at 6:18

• Changed state: Triaged

[GoogleCodeExporter]

Original comment by fry@google.com on 10 Dec 2011 at 3:36

• Added labels: Package-Collect

[GoogleCodeExporter]

Original comment by fry@google.com on 16 Feb 2012 at 7:17

• Changed state: Acknowledged

[GoogleCodeExporter]

Original comment by kevinb@google.com on 30 May 2012 at 7:43

• Added labels: Type-Addition
• Removed labels: Type-Enhancement

[GoogleCodeExporter]

Original comment by kevinb@google.com on 22 Jun 2012 at 6:16

• Changed state: Research

[GoogleCodeExporter]

Issue 1195 has been merged into this issue.

Original comment by lowas...@google.com on 7 Nov 2012 at 8:21

[GoogleCodeExporter]

Related to this I want to extend the RegularImmutableBiMap,
but there is no public class access.

Original comment by ame...@gmail.com on 10 Jan 2013 at 2:02

[GoogleCodeExporter]

The immutable collections can definitely not be extensible.  That's part of 
Effective Java item 15 -- immutable classes cannot be extended.

Original comment by lowas...@google.com on 10 Jan 2013 at 5:01

[GoogleCodeExporter]

This issue has been migrated to GitHub.

It can be found at https://github.com/google/guava/issues/<id>

Original comment by cgdecker@google.com on 1 Nov 2014 at 4:16

• Added labels: MigratedToGitHub

[GoogleCodeExporter]

Original comment by cgdecker@google.com on 1 Nov 2014 at 4:19

• Changed state: Migrated

[GoogleCodeExporter]

Original comment by cgdecker@google.com on 3 Nov 2014 at 9:10

• Added labels: Restrict-AddIssueComment-Commit