Java: Fix comment re. neutrals and WithoutElement.

The remove methods should not have been in this section, as they're
plain neutrals.

Author: aschackmull

java/ql/lib/ext/java.util.model.yml

@@ -430,6 +430,7 @@ extensions:
       - ["java.util", "Collection", "contains", "(Object)", "summary", "manual"]
       - ["java.util", "Collection", "containsAll", "(Collection)", "summary", "manual"]
       - ["java.util", "Collection", "isEmpty", "()", "summary", "manual"]
+      - ["java.util", "Collection", "removeIf", "(Predicate)", "summary", "manual"]
       - ["java.util", "Collection", "size", "()", "summary", "manual"]
       - ["java.util", "Collections", "emptyList", "()", "summary", "manual"]
       - ["java.util", "Collections", "emptyMap", "()", "summary", "manual"]
@@ -451,12 +452,14 @@ extensions:
       - ["java.util", "HashMap", "size", "()", "summary", "manual"]
       - ["java.util", "HashSet", "HashSet", "(int)", "summary", "manual"]
       - ["java.util", "Iterator", "hasNext", "()", "summary", "manual"]
+      - ["java.util", "Iterator", "remove", "()", "summary", "manual"]
       - ["java.util", "List", "contains", "(Object)", "summary", "manual"]
       - ["java.util", "List", "equals", "(Object)", "summary", "manual"]
       - ["java.util", "List", "hashCode", "()", "summary", "manual"]
       - ["java.util", "List", "indexOf", "(Object)", "summary", "manual"]
       - ["java.util", "List", "isEmpty", "()", "summary", "manual"]
       - ["java.util", "List", "of", "()", "summary", "manual"]
+      - ["java.util", "List", "remove", "(Object)", "summary", "manual"]
       - ["java.util", "List", "sort", "(Comparator)", "summary", "manual"]
       - ["java.util", "List", "size", "()", "summary", "manual"]
       - ["java.util", "Locale$Builder", "addUnicodeLocaleAttribute", "(String)", "summary", "df-manual"]
@@ -535,6 +538,8 @@ extensions:
       - ["java.util", "Scanner", "locale", "()", "summary", "df-manual"]
       - ["java.util", "Set", "contains", "(Object)", "summary", "manual"]
       - ["java.util", "Set", "isEmpty", "()", "summary", "manual"]
+      - ["java.util", "Set", "remove", "(Object)", "summary", "manual"]
+      - ["java.util", "Set", "removeAll", "(Collection)", "summary", "manual"]
       - ["java.util", "Set", "size", "()", "summary", "manual"]
       - ["java.util", "TreeMap", "TreeMap", "(Comparator)", "summary", "df-manual"]
       - ["java.util", "TreeSet", "TreeSet", "(Comparator)", "summary", "df-manual"]
@@ -545,14 +550,8 @@ extensions:
       - ["java.util", "TimeZone", "getTimeZone", "(String)", "summary", "manual"]
       - ["java.util", "Vector", "size", "()", "summary", "manual"]
 
-      # The below APIs are currently being stored as neutral models since `WithoutElement` has not yet been implemented for Java.
-      # When `WithoutElement` is implemented, these should be changed to summary models of the form `Argument[this].WithoutElement -> Argument[this]`.
-      - ["java.util", "Collection", "removeIf", "(Predicate)", "summary", "manual"]
-      - ["java.util", "Iterator", "remove", "()", "summary", "manual"]
-      - ["java.util", "List", "remove", "(Object)", "summary", "manual"]
+      # The below API is currently being stored as neutral models since `WithoutElement` does not yet have a counterpart for MapValue/MapKey.
       - ["java.util", "Map", "clear", "()", "summary", "manual"]
-      - ["java.util", "Set", "remove", "(Object)", "summary", "manual"]
-      - ["java.util", "Set", "removeAll", "(Collection)", "summary", "manual"]
 
       # The below APIs have numeric flow and are currently being stored as neutral models.
       # These may be changed to summary models with kinds "value-numeric" and "taint-numeric" (or similar) in the future.