Removed -Ynotnull setting.

It never worked and it would periodically jump out and bite
someone. Slash and burn so new plants can take root. Eliminated
NotNullType and NotNullClass, internal elements. Removed notNull
method on Type, which no longer has a purpose. Retained 'def
isNotNull' because there are types which are known by construction
not to be null (ThisType, ConstantType as long as the constant
isn't null, etc.) and that's still useful to know.

1 files changed, 11 insertions, 10 deletions

diff --git a/src/compiler/scala/tools/nsc/ast/TreeGen.scala b/src/compiler/scala/tools/nsc/ast/TreeGen.scala
index b9eb511a9a..d195d195af 100644
--- a/src/compiler/scala/tools/nsc/ast/TreeGen.scala
+++ b/src/compiler/scala/tools/nsc/ast/TreeGen.scala
@@ -19,16 +19,17 @@ abstract class TreeGen extends scala.reflect.internal.TreeGen with TreeDSL {
   import global._
   import definitions._
 
-  def mkCheckInit(tree: Tree): Tree = {
-    val tpe =
-      if (tree.tpe != null || !tree.hasSymbolField) tree.tpe
-      else tree.symbol.tpe
-
-    if (!global.phase.erasedTypes && settings.warnSelectNullable.value &&
-        tpe <:< NotNullClass.tpe && !tpe.isNotNull)
-      mkRuntimeCall(nme.checkInitialized, List(tree))
-    else
-      tree
+  def mkCheckInit(tree: Tree): Tree = { tree
+    // Leaving in the next commit...
+    // val tpe =
+    //   if (tree.tpe != null || !tree.hasSymbolField) tree.tpe
+    //   else tree.symbol.tpe
+
+    // if (!global.phase.erasedTypes && settings.warnSelectNullable.value &&
+    //     tpe <:< NotNullClass.tpe && !tpe.isNotNull)
+    //   mkRuntimeCall(nme.checkInitialized, List(tree))
+    // else
+    //   tree
   }
 
   /** Builds a fully attributed wildcard import node.