JS stdlib: drop wrong annotations in javautilCollectionsCode.kt and fix Collections.reverse.

js/js.libraries/src/core/javautilCollectionsCode.kt

@@ -3,30 +3,25 @@ package java.util.Collections
 import java.lang.*
 import java.util.*
 
+// TODO write tests for empty*
+
 // TODO should be immutable!
-library
 public val emptyList: List<Any> = ArrayList<Any>()
-library
 public val emptyMap: Map<Any, Any> = HashMap<Any,Any>()
 
-library
 public val <T> EMPTY_LIST: List<T>
     get() = emptyList<T>()
 
-library
 public val <K,V> EMPTY_MAP: Map<K,V>
     get() = emptyMap<K,V>()
 
-library
 public fun <T> emptyList(): List<T> = emptyList as List<T>
-library
 public fun <K,V> emptyMap(): Map<K,V> = emptyMap as Map<K,V>
 
-library
 public fun <T> reverse(list: MutableList<T>): Unit {
     val size = list.size()
-    for (i in 0.rangeTo(size / 2)) {
+    for (i in 0..(size / 2) - 1) {
-        val i2 = size - i
+        val i2 = size - i - 1
         val tmp = list[i]
         list[i] = list[i2]
         list[i2] = tmp