search

yangxu998 / guava-libraries

Automatically exported from code.google.com/p/guava-libraries
Apache License 2.0
0 stars 0 forks source link

ImmutableSortedMap should implement NavigableMap #664

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I understand that Guava doesn't support NavigableMap because of the requirement 
to retain Java5 backwards-compatiblity, but now that JDK7 is due out in 10 days 
I am hoping you will revisit the matter.

Either way, please keep this issue open for the purpose of collecting votes.

Original issue reported on code.google.com by cow...@bbs.darktech.org on 18 Jul 2011 at 8:34

GoogleCodeExporter commented 9 years ago 
No point in collecting votes on this issue; you want issue 32.

Original comment by kevinb@google.com on 19 Jul 2011 at 12:18

GoogleCodeExporter commented 9 years ago

Original comment by fry@google.com on 28 Jul 2011 at 5:23

GoogleCodeExporter commented 9 years ago 
That said, it already does implement the NavigableMap *methods* internally, but 
it a) doesn't expose them and b) doesn't implement the NavigableMap interface.  
If we exposed these, would that satisfy your needs?

Original comment by wasserman.louis on 16 Oct 2011 at 11:38

GoogleCodeExporter commented 9 years ago 
Exposing the methods would help, but I'd like to keep this issue open until the 
class implements the NavigableMap interface.

Original comment by cow...@bbs.darktech.org on 17 Oct 2011 at 1:56

GoogleCodeExporter commented 9 years ago 
Agreed, clearly.  I'm just not at all sure what kind of timeline we should 
expect for JDK 6.

Original comment by wasserman.louis on 17 Oct 2011 at 1:58

GoogleCodeExporter commented 9 years ago

Original comment by fry@google.com on 10 Dec 2011 at 4:12

GoogleCodeExporter commented 9 years ago 
I understand that Chris is working on this for release 12, but I'm not positive 
-- Chris?

Original comment by wasserman.louis on 17 Mar 2012 at 7:42

GoogleCodeExporter commented 9 years ago 
Correct.  I got an LGTM for "my" (really Louis's) CL for "ImmutableSortedSet 
implements NavigableSet," so I'll be submitting it on Monday.  After that, I 
don't anticipate any major problems with his ImmutableSortedMap CL.

Original comment by cpov...@google.com on 17 Mar 2012 at 8:11

GoogleCodeExporter commented 9 years ago 
Submitted internally.  Will be mirrored out soon.

Original comment by cpov...@google.com on 28 Mar 2012 at 7:41

GoogleCodeExporter commented 9 years ago 
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:15

GoogleCodeExporter commented 9 years ago

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