remove some unnecessary !! from stdlib

jdk-annotations/java/util/concurrent/annotations.xml

@@ -100,6 +100,9 @@
     <item name='java.util.concurrent.ExecutorService java.util.concurrent.Future&lt;T&gt; submit(java.util.concurrent.Callable&lt;T&gt;) 0'>
         <annotation name='org.jetbrains.annotations.NotNull'/>
     </item>
+    <item name='java.util.concurrent.ExecutorService java.util.concurrent.Future&lt;T&gt; submit(java.util.concurrent.Callable&lt;T&gt;)'>
+        <annotation name='org.jetbrains.annotations.NotNull'/>
+    </item>
     <item name='java.util.concurrent.ExecutorService java.util.concurrent.Future&lt;T&gt; submit(java.lang.Runnable, T) 0'>
         <annotation name='org.jetbrains.annotations.NotNull'/>
     </item>

libraries/stdlib/src/kotlin/concurrent/Thread.kt

@@ -6,10 +6,10 @@ import java.util.concurrent.Future
 import java.util.concurrent.Callable
 
 inline val currentThread : Thread
-    get() = Thread.currentThread()!!
+    get() = Thread.currentThread()
 
 inline var Thread.name : String
-    get() = getName()!!
+    get() = getName()
     set(name: String) { setName(name) }
 
 inline var Thread.daemon : Boolean
@@ -66,7 +66,7 @@ public inline fun Executor.invoke(action: ()->Unit) {
  */
 public inline fun <T>ExecutorService.submit(action: ()->T):Future<T> {
     val c:Callable<T> = callable(action)
-    return submit(c)!!;
+    return submit(c);
 }
 
 /**

libraries/stdlib/src/kotlin/nullable/Nullables.kt

@@ -129,7 +129,7 @@ public inline fun <T> T?.makeString(separator: String = ", ", prefix: String = "
         buffer.append(this)
     }
     buffer.append(postfix)
-    return buffer.toString()!!
+    return buffer.toString()
 }
 
 

libraries/stdlib/src/kotlin/support/AbstractIterator.kt

@@ -108,7 +108,7 @@ class CompositeIterator<T>(vararg iterators: Iterator<T>): AbstractIterator<T>()
             val iter = currentIter
             if (iter != null) {
                 if (iter.hasNext()) {
-                    setNext(iter.next()!!)
+                    setNext(iter.next())
                     return
                 } else {
                     currentIter = null

libraries/stdlib/src/kotlin/template/Templates.kt

@@ -116,7 +116,7 @@ public open class ToStringFormatter : Formatter {
     }
 }
 
-public val defaultLocale : Locale = Locale.getDefault()!!
+public val defaultLocale : Locale = Locale.getDefault()
 
 /**
  * Formats values using a given [[Locale]] for internationalisation

libraries/stdlib/test/CollectionTest.kt

@@ -409,7 +409,7 @@ class CollectionTest {
         private val collection = collection
 
         override fun iterator(): Iterator<T> {
-            return collection.iterator()!!
+            return collection.iterator()
         }
     }
 }

libraries/stdlib/test/IoTest.kt

@@ -73,7 +73,7 @@ class IoTest() : TestCase() {
   }
   
   fun testListFiles() {
-    val dir = File.createTempFile("temp", System.nanoTime().toString())!!
+    val dir = File.createTempFile("temp", System.nanoTime().toString())
     dir.delete()
     dir.mkdir()
     
@@ -81,7 +81,7 @@ class IoTest() : TestCase() {
     File.createTempFile("temp", "2.java", dir)
     File.createTempFile("temp", "3.kt", dir)
     
-    val result = dir.listFiles { it.getName()!!.endsWith(".kt") }
+    val result = dir.listFiles { it.getName().endsWith(".kt") }
     
     assertNotNull(result)
     assertEquals(result!!.size, 2)

libraries/stdlib/test/concurrent/ThreadTest.kt

@@ -11,7 +11,7 @@ import java.util.concurrent.TimeUnit.*
 class ThreadTest {
     test fun scheduledTask() {
 
-        val pool = Executors.newFixedThreadPool(1)!!
+        val pool = Executors.newFixedThreadPool(1)
         val countDown = CountDownLatch(1)
         pool {
             countDown.countDown()
@@ -21,7 +21,7 @@ class ThreadTest {
 
     test fun callableInvoke() {
 
-        val pool = Executors.newFixedThreadPool(1)!!
+        val pool = Executors.newFixedThreadPool(1)
         val future = pool<String> {
             "Hello"
         }

libraries/stdlib/test/regressions/Kt2495Test.kt

@@ -14,7 +14,7 @@ fun f(xs: Iterator<Int>): Int {
 class Kt2495Test {
     test fun duplicateIteratorsBug() {
         val list = arrayList(1, 2, 3)
-        val result = f(list.iterator()!!)
+        val result = f(list.iterator())
         assertEquals(6, result)
     }
 }

libraries/stdlib/test/regressions/kt1202.kt

@@ -99,7 +99,7 @@ fun parseAtomic(tokens : Deque<Token>) : ParseResult<Expression> {
             else
                 Failure("Expecting ')'")
         }
-        is Number -> Success(Num(Integer.parseInt((token as Token).text)!!))
+        is Number -> Success(Num(Integer.parseInt((token as Token).text)))
         else -> Failure("Unexpected EOF")
     }
 }

libraries/stdlib/test/template/LocaleTemplateTest.kt

@@ -13,11 +13,11 @@ class LocaleTemplateTest : TestCase() {
     }
 
     fun testFrance() : Unit {
-        format(LocaleFormatter(Locale.FRANCE!!))
+        format(LocaleFormatter(Locale.FRANCE))
     }
 
     fun testGermany() : Unit {
-        format(LocaleFormatter(Locale.GERMANY!!))
+        format(LocaleFormatter(Locale.GERMANY))
     }
 
     fun format(formatter: LocaleFormatter): Unit {