Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8029055

Map.merge must refuse null values

    XMLWordPrintable

Details

    Backports

      Description

        Map.merge<http://download.java.net/jdk8/docs/api/java/util/Map.html#merge-K-V-java.util.function.BiFunction-&gt;states
        that its default implementation is equivalent to:

        V oldValue = map.get(key);
         V newValue = (oldValue == null) ? value :
                      remappingFunction.apply(oldValue, value);
         if (newValue == null)
             map.remove(key);
         else if (oldValue == null)
             map.remove(key);
         else
             map.put(key, newValue);

        This is not correct as for non-existing or null-value, the non-null newValuw should be added to the map.

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8030348 Map.merge must refuse null values

            • P2 - Crashes, loss of data, severe memory leak.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8031288 Map.merge must refuse null values

            • P2 - Crashes, loss of data, severe memory leak.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8031011 Map.merge must refuse null values

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed
            relates to

            Bug - A problem which impairs or prevents the functions of the product. JDK-8292955 Collections.checkedMap Map.merge does not properly check key and value

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Open

            Activity

              People

                mduigou Mike Duigou
                henryjen Henry Jen
                Votes:
                0 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: