Revert "Restore default parameter values in JS collection constructors"

This reverts commit 0f972f24

libraries/stdlib/api/js-v1/kotlin.collections.kt

@@ -10461,7 +10461,7 @@ public final class ArrayDeque<E> : kotlin.collections.AbstractMutableList<E> {
 public open class ArrayList<E> : kotlin.collections.AbstractMutableList<E>, kotlin.collections.MutableList<E>, kotlin.collections.RandomAccess {
     public constructor ArrayList<E>()
 
-    public constructor ArrayList<E>(initialCapacity: kotlin.Int = ...)
+    public constructor ArrayList<E>(initialCapacity: kotlin.Int)
 
     public constructor ArrayList<E>(elements: kotlin.collections.Collection<E>)
 
@@ -10618,7 +10618,7 @@ public open class HashMap<K, V> : kotlin.collections.AbstractMutableMap<K, V>, k
 
     public constructor HashMap<K, V>(initialCapacity: kotlin.Int)
 
-    public constructor HashMap<K, V>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float = ...)
+    public constructor HashMap<K, V>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float)
 
     public constructor HashMap<K, V>(original: kotlin.collections.Map<out K, V>)
 
@@ -10644,7 +10644,7 @@ public open class HashSet<E> : kotlin.collections.AbstractMutableSet<E>, kotlin.
 
     public constructor HashSet<E>(initialCapacity: kotlin.Int)
 
-    public constructor HashSet<E>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float = ...)
+    public constructor HashSet<E>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float)
 
     public constructor HashSet<E>(elements: kotlin.collections.Collection<E>)
 
@@ -10724,7 +10724,7 @@ public open class LinkedHashMap<K, V> : kotlin.collections.HashMap<K, V>, kotlin
 
     public constructor LinkedHashMap<K, V>(initialCapacity: kotlin.Int)
 
-    public constructor LinkedHashMap<K, V>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float = ...)
+    public constructor LinkedHashMap<K, V>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float)
 
     public constructor LinkedHashMap<K, V>(original: kotlin.collections.Map<out K, V>)
 
@@ -10748,7 +10748,7 @@ public open class LinkedHashSet<E> : kotlin.collections.HashSet<E>, kotlin.colle
 
     public constructor LinkedHashSet<E>(initialCapacity: kotlin.Int)
 
-    public constructor LinkedHashSet<E>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float = ...)
+    public constructor LinkedHashSet<E>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float)
 
     public constructor LinkedHashSet<E>(elements: kotlin.collections.Collection<E>)
 }

libraries/stdlib/api/js/kotlin.collections.kt

@@ -10425,7 +10425,7 @@ public final class ArrayDeque<E> : kotlin.collections.AbstractMutableList<E> {
 public open class ArrayList<E> : kotlin.collections.AbstractMutableList<E>, kotlin.collections.MutableList<E>, kotlin.collections.RandomAccess {
     public constructor ArrayList<E>()
 
-    public constructor ArrayList<E>(initialCapacity: kotlin.Int = ...)
+    public constructor ArrayList<E>(initialCapacity: kotlin.Int)
 
     public constructor ArrayList<E>(elements: kotlin.collections.Collection<E>)
 
@@ -10530,7 +10530,7 @@ public open class HashMap<K, V> : kotlin.collections.AbstractMutableMap<K, V>, k
 
     public constructor HashMap<K, V>(initialCapacity: kotlin.Int)
 
-    public constructor HashMap<K, V>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float = ...)
+    public constructor HashMap<K, V>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float)
 
     public constructor HashMap<K, V>(original: kotlin.collections.Map<out K, V>)
 
@@ -10556,7 +10556,7 @@ public open class HashSet<E> : kotlin.collections.AbstractMutableSet<E>, kotlin.
 
     public constructor HashSet<E>(initialCapacity: kotlin.Int)
 
-    public constructor HashSet<E>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float = ...)
+    public constructor HashSet<E>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float)
 
     public constructor HashSet<E>(elements: kotlin.collections.Collection<E>)
 
@@ -10618,7 +10618,7 @@ public open class LinkedHashMap<K, V> : kotlin.collections.HashMap<K, V>, kotlin
 
     public constructor LinkedHashMap<K, V>(initialCapacity: kotlin.Int)
 
-    public constructor LinkedHashMap<K, V>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float = ...)
+    public constructor LinkedHashMap<K, V>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float)
 
     public constructor LinkedHashMap<K, V>(original: kotlin.collections.Map<out K, V>)
 
@@ -10642,7 +10642,7 @@ public open class LinkedHashSet<E> : kotlin.collections.HashSet<E>, kotlin.colle
 
     public constructor LinkedHashSet<E>(initialCapacity: kotlin.Int)
 
-    public constructor LinkedHashSet<E>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float = ...)
+    public constructor LinkedHashSet<E>(initialCapacity: kotlin.Int, loadFactor: kotlin.Float)
 
     public constructor LinkedHashSet<E>(elements: kotlin.collections.Collection<E>)
 }

libraries/stdlib/js/src/kotlin/collections/ArrayList.kt

@@ -24,8 +24,7 @@ public actual open class ArrayList<E> internal constructor(private var array: Ar
      * Creates an empty [ArrayList].
      * @param initialCapacity initial capacity (ignored)
      */
-    @Suppress("ACTUAL_FUNCTION_WITH_DEFAULT_ARGUMENTS")
-    public actual constructor(initialCapacity: Int = 0) : this(emptyArray()) {}
+    public actual constructor(initialCapacity: Int) : this(emptyArray()) {}
 
     /**
      * Creates an [ArrayList] filled from the [elements] collection.

libraries/stdlib/js/src/kotlin/collections/HashMap.kt

@@ -69,8 +69,7 @@ public actual open class HashMap<K, V> : AbstractMutableMap<K, V>, MutableMap<K,
      *
      * @throws IllegalArgumentException if the initial capacity or load factor are negative
      */
-    @Suppress("ACTUAL_FUNCTION_WITH_DEFAULT_ARGUMENTS")
-    actual constructor(initialCapacity: Int, loadFactor: Float = 0.0f) : this() {
+    actual constructor(initialCapacity: Int, loadFactor: Float) : this() {
         // This implementation of HashMap has no need of load factors or capacities.
         require(initialCapacity >= 0) { "Negative initial capacity: $initialCapacity" }
         require(loadFactor >= 0) { "Non-positive load factor: $loadFactor" }

libraries/stdlib/js/src/kotlin/collections/HashSet.kt

@@ -41,8 +41,7 @@ public actual open class HashSet<E> : AbstractMutableSet<E>, MutableSet<E> {
      *
      * @throws IllegalArgumentException if the initial capacity or load factor are negative
      */
-    @Suppress("ACTUAL_FUNCTION_WITH_DEFAULT_ARGUMENTS")
-    actual constructor(initialCapacity: Int, loadFactor: Float = 0.0f) {
+    actual constructor(initialCapacity: Int, loadFactor: Float) {
         map = HashMap<E, Any>(initialCapacity, loadFactor)
     }
 

libraries/stdlib/js/src/kotlin/collections/LinkedHashMap.kt

@@ -186,8 +186,7 @@ public actual open class LinkedHashMap<K, V> : HashMap<K, V>, MutableMap<K, V> {
      *
      * @throws IllegalArgumentException if the initial capacity or load factor are negative
      */
-    @Suppress("ACTUAL_FUNCTION_WITH_DEFAULT_ARGUMENTS")
-    actual constructor(initialCapacity: Int, loadFactor: Float = 0.0f) : super(initialCapacity, loadFactor) {
+    actual constructor(initialCapacity: Int, loadFactor: Float) : super(initialCapacity, loadFactor) {
         map = HashMap<K, ChainEntry<K, V>>()
     }
 

libraries/stdlib/js/src/kotlin/collections/LinkedHashSet.kt

@@ -38,8 +38,7 @@ public actual open class LinkedHashSet<E> : HashSet<E>, MutableSet<E> {
      *
      * @throws IllegalArgumentException if the initial capacity or load factor are negative
      */
-    @Suppress("ACTUAL_FUNCTION_WITH_DEFAULT_ARGUMENTS")
-    actual constructor(initialCapacity: Int, loadFactor: Float = 0.0f) : super(LinkedHashMap<E, Any>(initialCapacity, loadFactor))
+    actual constructor(initialCapacity: Int, loadFactor: Float) : super(LinkedHashMap<E, Any>(initialCapacity, loadFactor))
 
     actual constructor(initialCapacity: Int) : this(initialCapacity, 0.0f)