Issue1816103

The attached patch adds methods to make org.python.core.PyDictionary a drop-in for wherever java.util.Map is required.

Complete implementation
File Added: jythonDictMap.pat

Oops! putAll was missing -- added it in. This is the last update for today, I promise.
File Added: jythonDictMap.pat

Some comments on the patch.
I think that it is a great idea to make the jython types more compatible with the common java types. This could be targeted in jython 2.5+, but it will probably follow with a number of other changes.


Some comments on the implementation in this patch:

This patch changes more than it is supposed to implement. What is the motivation behind making all parameters final?

containsValue : should, as the name implies, check if the Map contains a mapping with the argument as value, i.e. not as key, which is was __contains__ checks for.

entrySet : should return a set of Map.Entry objects, that is objects that contains both the key and the value. So we will probably use a loop to create these, or return our own specific set that creates these objects on the fly.

keySet : should return a set of the keys in the Map, the same as dict.keys() in python.

put: should return the old value associated with the key, so you'll need to __getitem__ it before you invoke __setitem__

putAll : this is actually quite simple to implement just iterate over arg0.entrySet() and add the key mapped to the value for each entry in that set.

remove : should remove the binding associated with the key, not associate it with null. Use __delitem__ here instead.

Another issue is that the Map methods do not correctly convert null values to Py.None.

I've created a patch that addresses these issues here:
https://sourceforge.net/tracker/index.php?func=detail&aid=1817565&group_id=12867&atid=312867

Byron's patch was applied in r3802.