Deprecation patrol. There was a bug among the deprecation

warnings due to my changing a map from mutable to immutable (which
ought to be the good direction) because "def update" still
lingers on immutable maps.  I counted the days elapsed since it
was marked for death (before 2.8.0) and added in the bugliness
of what I was looking at and bid it farewell.

Now removed: def update on immutable maps.  No review.

git-svn-id: http://lampsvn.epfl.ch/svn-repos/scala/scala/trunk@24469 5e8d7ff9-d8ef-0310-90f0-a4852d11357a

Author: extempore

src/compiler/scala/tools/nsc/CompileServer.scala

@@ -180,6 +180,6 @@ object CompileServer extends StandardCompileServer {
     run()
 
     compileSocket.deletePort(port)
-    exit(0)
+    sys.exit(0)
   }
 }

src/compiler/scala/tools/nsc/MainGenericRunner.scala

@@ -73,8 +73,7 @@ object MainGenericRunner {
     else command.thingToRun match {
       case None             =>
         // We start the repl when no arguments are given.
-        new ILoop main settings
-        true  // not actually reached in general
+        new ILoop process settings
 
       case Some(thingToRun) =>
         val isObjectName =

src/compiler/scala/tools/nsc/doc/html/page/Template.scala

@@ -63,13 +63,13 @@ class Template(tpl: DocTemplateEntity) extends HtmlPage {
 
         <div id="mbrsel">
           <div id='textfilter'><span class='pre'/><span class='input'><input type='text' accesskey='/'/></span><span class='post'/></div>
-          { if (tpl.linearization.isEmpty) NodeSeq.Empty else
+          { if (tpl.linearizationTemplates.isEmpty) NodeSeq.Empty else
               <div id="order">
                 <span class="filtertype">Ordering</span>
                 <ol><li class="alpha in">Alphabetic</li><li class="inherit out">By inheritance</li></ol>
               </div>
           }
-          { if (tpl.linearization.isEmpty) NodeSeq.Empty else
+          { if (tpl.linearizationTemplates.isEmpty) NodeSeq.Empty else
               <div id="ancestors">
                 <span class="filtertype">Inherited</span>
                 <ol><li class="hideall">Hide All</li><li class="showall">Show all</li></ol>
@@ -112,7 +112,7 @@ class Template(tpl: DocTemplateEntity) extends HtmlPage {
         }
 
         {
-          NodeSeq fromSeq (for ((superTpl, superType) <- tpl.linearization) yield
+          NodeSeq fromSeq (for ((superTpl, superType) <- (tpl.linearizationTemplates zip tpl.linearizationTypes)) yield
             <div class="parent" name={ superTpl.qualifiedName }>
               <h3>Inherited from {
                 if (tpl.universe.settings.useStupidTypes.value)
@@ -292,7 +292,7 @@ class Template(tpl: DocTemplateEntity) extends HtmlPage {
       }
     } ++
     { mbr match {
-        case dtpl: DocTemplateEntity if (isSelf && !dtpl.linearization.isEmpty && !isReduced) =>
+        case dtpl: DocTemplateEntity if (isSelf && !dtpl.linearizationTemplates.isEmpty && !isReduced) =>
           <div class="block">
             linear super types: { typesToHtml(dtpl.linearizationTypes, hasLinks = true, sep = xml.Text(", ")) }
           </div>

src/compiler/scala/tools/nsc/interpreter/ILoop.scala

@@ -734,10 +734,9 @@ class ILoop(in0: Option[BufferedReader], protected val out: PrintWriter)
 
     // if they asked for no help and command is valid, we call the real main
     neededHelp() match {
-      case ""     => if (command.ok) main(command.settings) // else nothing
-      case help   => plush(help)
+      case ""     => command.ok && process(command.settings)
+      case help   => plush(help) ; true
     }
-    true
   }
 
   @deprecated("Use `process` instead")
@@ -763,7 +762,7 @@ object ILoop {
         val settings = new Settings
         settings.classpath.value = sys.props("java.class.path")
 
-        repl main settings
+        repl process settings
       }
     }
   }

src/compiler/scala/tools/nsc/symtab/clr/TypeParser.scala

@@ -92,7 +92,12 @@ abstract class TypeParser {
     val flags = Flags.JAVA | Flags.STATIC | Flags.IMPLICIT; // todo: static? shouldn't be final instead?
     val viewMethodType = (msym: Symbol) => JavaMethodType(msym.newSyntheticValueParams(List(fromTpe)), toTpe)
     val vmsym = createMethod(nme.view_ + viewSuffix, flags, viewMethodType, null, true);
-    if (addToboxMethodMap) definitions.boxMethod(clazz) = vmsym
+    // !!! this used to mutate a mutable map in definitions, but that map became
+    // immutable and this kept "working" with a no-op.  So now it's commented out
+    // since I retired the deprecated code which allowed for that bug.
+    //
+    // if (addToboxMethodMap) definitions.boxMethod(clazz) = vmsym
+    
     if (isAddressOf) clrTypes.addressOfViews += vmsym
     vmsym
   }

src/library/scala/collection/immutable/IntMap.scala

@@ -269,9 +269,6 @@ sealed abstract class IntMap[+T] extends Map[Int, T] with MapLike[Int, T, IntMap
     case IntMap.Nil => IntMap.Tip(key, value);
   }
 
-  @deprecated("use `updated' instead")
-  override def update[S >: T](key: Int, value: S): IntMap[S] = updated(key, value)
-
   /**
    * Updates the map, using the provided function to resolve conflicts if the key is already present.
    * 

src/library/scala/collection/immutable/LongMap.scala

@@ -267,9 +267,6 @@ sealed abstract class LongMap[+T] extends Map[Long, T] with MapLike[Long, T, Lon
     case LongMap.Nil => LongMap.Tip(key, value);
   }
 
-  @deprecated("use `updated' instead")
-  override def update[S >: T](key: Long, value: S): LongMap[S] = updated(key, value)
-
   /**
    * Updates the map, using the provided function to resolve conflicts if the key is already present.
    * 

src/library/scala/collection/immutable/MapLike.scala

@@ -6,8 +6,6 @@
 **                          |/                                          **
 \*                                                                      */
 
-
-
 package scala.collection
 package immutable
 
@@ -124,7 +122,5 @@ trait MapLike[A, +B, +This <: MapLike[A, B, This] with Map[A, B]]
     for ((key, value) <- this) b += ((key, f(key, value)))
     b.result
   }
-
-  @deprecated("use `updated' instead")
-  def update[B1 >: B](key: A, value: B1): immutable.Map[A, B1] = updated(key, value).asInstanceOf[immutable.Map[A, B1]]
 }
+

src/library/scala/collection/immutable/TrieIteratorBase.scala

@@ -168,7 +168,7 @@ private[immutable] abstract class TrieIteratorBase[+T, CC >: Null <: Iterable[T]
           val arr: Array[CC] = determineType(m) match {
             case COLLISION_TYPE => collisionToArray(m)
             case TRIE_TYPE      => getElems(m)
-            case _              => error("cannot divide single element")
+            case _              => sys.error("cannot divide single element")
           }
           arrayToIterators(arr)
         }

src/partest/scala/tools/partest/utils/Properties.scala

@@ -13,5 +13,5 @@ package utils
 /** Loads partest.properties from the jar. */
 object Properties extends scala.util.PropertiesTrait {
   protected def propCategory    = "partest"
-  protected def pickJarBasedOn  = classOf[Application]
+  protected def pickJarBasedOn  = classOf[nest.Worker]
 }