[ https://issues.apache.org/jira/browse/HARMONY-5788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12593280#action_12593280
]
Mark Hindess commented on HARMONY-5788:
---------------------------------------
I agree that refactoring might be helpful but unless I'm missing something then I think we
still need to check that v2 == null, so something like:
return v1 == null ? ( v2 == null && TreeMap.this.containsKey(key) ) : v1.equals(v2);
> [classlib][util] TreeMap.entrySet().contains() returns true for entries with null value
> ---------------------------------------------------------------------------------------
>
> Key: HARMONY-5788
> URL: https://issues.apache.org/jira/browse/HARMONY-5788
> Project: Harmony
> Issue Type: Bug
> Components: Classlib
> Reporter: Mark Hindess
>
> The following code illustrates the problem:
> import java.util.TreeMap;
> public class TestTreeMap {
> public static void main(String[] args) {
> TreeMap master = new TreeMap<String, String>();
> master.put("one", "1");
> master.put("null", null);
> TreeMap map = new TreeMap<String, String>();
> Object[] element = master.entrySet().toArray();
> for (int i = 0; i < element.length ; i++) {
> System.out.println("Contains entry " + element[i] + "? == " +
> map.entrySet().contains(element[i]));
> }
> }
> }
> On harmony, the output is:
> Contains entry null=null? == true
> Contains entry one=1? == false
> on the RI, the output is:
> Contains entry null=null? == false
> Contains entry one=1? == false
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
|