From 3bb8745ca7dcda8c81103b3965b83973b4a72214 Mon Sep 17 00:00:00 2001
From: Som Snytt <som.snytt@gmail.com>
Date: Thu, 10 Jan 2013 13:33:45 -0800
Subject: Fixes and features for javap (fixing SI-6894)

Output filtering is refactored for javap6.

That means javap6 also supports -raw.

Handling of # is: Foo#foo filter on foo,
Foo# filter on apply, -fun Foo#foo for anonfuns of foo,
-fun Foo# anonfuns filtering on apply.

One is loath to add command options, so it's not possible
to ask for "only apply methods in anonfuns pertaining to
a method."  Hypothetical syntax to say "show me the
apply only": -fun Foo#foo(), for future reference.
---
 src/compiler/scala/tools/util/Javap.scala          | 165 +++++++++++++--------
 test/files/run/repl-javap-outdir-funs/foo_1.scala  |   6 +
 .../run/repl-javap-outdir-funs/run-repl_7.scala    |  12 ++
 3 files changed, 118 insertions(+), 65 deletions(-)
 create mode 100644 test/files/run/repl-javap-outdir-funs/foo_1.scala
 create mode 100644 test/files/run/repl-javap-outdir-funs/run-repl_7.scala

diff --git a/src/compiler/scala/tools/util/Javap.scala b/src/compiler/scala/tools/util/Javap.scala
index b39d5a5d89..cbfd8fec51 100644
--- a/src/compiler/scala/tools/util/Javap.scala
+++ b/src/compiler/scala/tools/util/Javap.scala
@@ -61,11 +61,11 @@ class JavapClass(
   def apply(args: Seq[String]): List[JpResult] = {
     val (options, claases) = args partition (s => (s startsWith "-") && s.length > 1)
     val (flags, upgraded) = upgrade(options)
-    if (flags.help || claases.isEmpty) List(JpResult(JavapTool.helper(printWriter)))
-    else {
-      val targets = if (flags.fun) FunFinder(loader, intp).funs(claases) else claases
-      tool(flags.raw, upgraded)(targets map (claas => claas -> bytesFor(claas, flags.app)))
-    }
+    import flags.{ app, fun, help, raw }
+    val targets = if (fun && !help) FunFinder(loader, intp).funs(claases) else claases
+    if (help || claases.isEmpty) List(JpResult(JavapTool.helper(printWriter)))
+    else if (targets.isEmpty) List(JpResult("No anonfuns found."))
+    else tool(raw, upgraded)(targets map (claas => claas -> bytesFor(claas, app)))
   }
 
   /** Cull our tool options. */
@@ -74,10 +74,14 @@ class JavapClass(
     case (t,s)                => (t, JavapTool.DefaultOptions)
   }
 
-  /** Find bytes. Handle "-", "-app", "Foo#bar" (by ignoring member). */
+  /** Find bytes. Handle "-", "-app", "Foo#bar" (by ignoring member), "#bar" (by taking "bar"). */
   private def bytesFor(path: String, app: Boolean) = Try {
     def last = intp.get.mostRecentVar  // fail if no intp
-    def req = if (path == "-") last else path.splitHashMember._1
+    def req = if (path == "-") last else {
+      val s = path.splitHashMember
+      if (s._1.nonEmpty) s._1
+      else s._2 getOrElse "#"
+    }
     def asAppBody(s: String) = {
       val (cls, fix) = s.splitSuffix
       s"${cls}$$delayedInit$$body${fix}"
@@ -146,16 +150,70 @@ class JavapClass(
     load(q)
   }
 
+  /** Base class for javap tool adapters for java 6 and 7. */
   abstract class JavapTool {
     type ByteAry = Array[Byte]
     type Input = Pair[String, Try[ByteAry]]
+
+    /** Run the tool. */
     def apply(raw: Boolean, options: Seq[String])(inputs: Seq[Input]): List[JpResult]
+
     // Since the tool is loaded by reflection, check for catastrophic failure.
     protected def failed: Boolean
     implicit protected class Failer[A](a: =>A) {
       def orFailed[B >: A](b: =>B) = if (failed) b else a
     }
     protected def noToolError = new JpError(s"No javap tool available: ${getClass.getName} failed to initialize.")
+
+    // output filtering support
+    val writer = new CharArrayWriter
+    def written = {
+      writer.flush()
+      val w = writer.toString
+      writer.reset()
+      w
+    }
+
+    /** Create a Showable with output massage.
+     *  @param raw show ugly repl names
+     *  @param target attempt to filter output to show region of interest
+     *  @param preamble other messages to output
+     */
+    def showWithPreamble(raw: Boolean, target: String, preamble: String = ""): Showable = new Showable {
+      // ReplStrippingWriter clips and scrubs on write(String)
+      // circumvent it by write(mw, 0, mw.length) or wrap it in withoutUnwrapping
+      def show() =
+        if (raw && intp.isDefined) intp.get withoutUnwrapping { writeLines() }
+        else writeLines()
+      private def writeLines() {
+        // take Foo# as Foo#apply for purposes of filtering. Useful for -fun Foo#;
+        // if apply is added here, it's for other than -fun: javap Foo#, perhaps m#?
+        val filterOn = target.splitHashMember._2 map { s => if (s.isEmpty) "apply" else s }
+        var filtering = false   // true if in region matching filter
+        // true to output
+        def checkFilter(line: String) = if (filterOn.isEmpty) true else {
+          // cheap heuristic, todo maybe parse for the java sig.
+          // method sigs end in paren semi
+          def isAnyMethod = line.endsWith(");")
+          def isOurMethod = {
+            val lparen = line.lastIndexOf('(')
+            val blank = line.lastIndexOf(' ', lparen)
+            (blank >= 0 && line.substring(blank+1, lparen) == filterOn.get)
+          }
+          filtering = if (filtering) {
+            // next blank line terminates section
+            // for -public, next line is next method, more or less
+            line.trim.nonEmpty && !isAnyMethod
+          } else {
+            isAnyMethod && isOurMethod
+          }
+          filtering
+        }
+        for (line <- Source.fromString(preamble + written).getLines; if checkFilter(line))
+          printWriter write line+lineSeparator
+        printWriter.flush()
+      }
+    }
   }
 
   class JavapTool6 extends JavapTool {
@@ -165,10 +223,13 @@ class JavapClass(
     override protected def failed = (EnvClass eq null) || (PrinterClass eq null)
 
     val PrinterCtr = PrinterClass.getConstructor(classOf[InputStream], classOf[PrintWriter], EnvClass) orFailed null
+    val printWrapper = new PrintWriter(writer)
     def newPrinter(in: InputStream, env: FakeEnvironment): FakePrinter =
-      PrinterCtr.newInstance(in, printWriter, env) orFailed null
-    def showable(fp: FakePrinter) = new Showable {
-      def show() = fp.asInstanceOf[{ def print(): Unit }].print()
+      PrinterCtr.newInstance(in, printWrapper, env) orFailed null
+    def showable(raw: Boolean, target: String, fp: FakePrinter): Showable = {
+      fp.asInstanceOf[{ def print(): Unit }].print()      // run tool and flush to buffer
+      printWrapper.flush()  // just in case
+      showWithPreamble(raw, target)
     }
 
     lazy val parser = new JpOptions
@@ -187,8 +248,8 @@ class JavapClass(
 
     override def apply(raw: Boolean, options: Seq[String])(inputs: Seq[Input]): List[JpResult] =
       (inputs map {
-        case (_, Success(ba)) => JpResult(showable(newPrinter(new ByteArrayInputStream(ba), newEnv(options))))
-        case (_, Failure(e))  => JpResult(e.toString)
+        case (claas, Success(ba)) => JpResult(showable(raw, claas, newPrinter(new ByteArrayInputStream(ba), newEnv(options))))
+        case (_, Failure(e))      => JpResult(e.toString)
       }).toList orFailed List(noToolError)
   }
 
@@ -284,50 +345,11 @@ class JavapClass(
           case _          => false
         }
     }
-    val writer = new CharArrayWriter
     def fileManager(inputs: Seq[Input]) = new JavapFileManager(inputs)()
-    def showable(raw: Boolean, target: String): Showable = {
-      val written = {
-        writer.flush()
-        val w = writer.toString
-        writer.reset()
-        w
-      }
-      val msgs = reporter.reportable(raw)
-      new Showable {
-        val mw = msgs + written
-        // ReplStrippingWriter clips and scrubs on write(String)
-        // circumvent it by write(mw, 0, mw.length) or wrap it in withoutUnwrapping
-        def show() =
-          if (raw && intp.isDefined) intp.get withoutUnwrapping { writeLines() }
-          else writeLines()
-        private def writeLines() {
-          // take Foo# as Foo#apply for purposes of filtering. Useful for -fun Foo#
-          val filterOn = target.splitHashMember._2 map { s => if (s.isEmpty) "apply" else s }
-          var filtering = false   // true if in region matching filter
-          // true to output
-          def checkFilter(line: String) = if (filterOn.isEmpty) true else {
-            def isOurMethod = {
-              val lparen = line.lastIndexOf('(')
-              val blank = line.lastIndexOf(' ', lparen)
-              (blank >= 0 && line.substring(blank+1, lparen) == filterOn.get)
-            }
-            filtering = if (filtering) {
-              // next blank line terminates section
-              line.trim.nonEmpty
-            } else {
-              // cheap heuristic, todo maybe parse for the java sig.
-              // method sigs end in paren semi
-              line.endsWith(");") && isOurMethod
-            }
-            filtering
-          }
-          for (line <- Source.fromString(mw).getLines; if checkFilter(line))
-            printWriter write line+lineSeparator
-          printWriter.flush()
-        }
-      }
-    }
+
+    // show tool messages and tool output, with output massage
+    def showable(raw: Boolean, target: String): Showable = showWithPreamble(raw, target, reporter.reportable(raw))
+
     // eventually, use the tool interface
     def task(options: Seq[String], claases: Seq[String], inputs: Seq[Input]): Task = {
       //ServiceLoader.load(classOf[javax.tools.DisassemblerTool]).
@@ -529,7 +551,11 @@ object JavapClass {
     /* only the file location from which the given class is loaded */
     def locate(k: String): Option[Path] = {
       Try {
-        (cl loadClass k).getProtectionDomain.getCodeSource.getLocation
+        val claas = try cl loadClass k catch {
+          case _: NoClassDefFoundError => null    // let it snow
+        }
+        // cf ScalaClassLoader.originOfClass
+        claas.getProtectionDomain.getCodeSource.getLocation
       } match {
         case Success(null)              => None
         case Success(loc) if loc.isFile => Some(Path(new JFile(loc.toURI)))
@@ -588,28 +614,37 @@ object JavapClass {
       (new Jar(f) map maybe).flatten
     }
     def loadable(name: String) = loader resourceable name
-    // translated class, optional member, whether it is repl output
-    def translate(s: String): (String, Option[String], Boolean) = {
+    // translated class, optional member, opt member to filter on, whether it is repl output
+    def translate(s: String): (String, Option[String], Option[String], Boolean) = {
       val (k0, m0) = s.splitHashMember
+      val k = k0.asClassName
       val member = m0 filter (_.nonEmpty)  // take Foo# as no member, not ""
+      val filter = m0 flatMap { case "" => Some("apply") case _ => None }   // take Foo# as filter on apply
       // class is either something replish or available to loader
       // $line.$read$$etc$Foo#member
-      ((intp flatMap (_ translatePath k0) filter (loadable) map ((_, member, true)))
-      // s = "f" and $line.$read$$etc$#f is what we're after, ignoring any #member
-      orElse (intp flatMap (_ translateEnclosingClass k0) map ((_, Some(s), true)))
-      getOrElse (k0, member, false))
+      ((intp flatMap (_ translatePath k) filter (loadable) map ((_, member, filter, true)))
+      // s = "f" and $line.$read$$etc$#f is what we're after,
+      // ignoring any #member (except take # as filter on #apply)
+      orElse (intp flatMap (_ translateEnclosingClass k) map ((_, Some(k), filter, true)))
+      getOrElse (k, member, filter, false))
     }
     /** Find the classnames of anonfuns associated with k,
      *  where k may be an available class or a symbol in scope.
      */
     def funsOf(k0: String): Seq[String] = {
       // class is either something replish or available to loader
-      val (k, member, isReplish) = translate(k0)
+      val (k, member, filter, isReplish) = translate(k0)
       val splat   = k split "\\."
       val name    = splat.last
       val prefix  = if (splat.length > 1) splat.init mkString "/" else ""
       val pkg     = if (splat.length > 1) splat.init mkString "." else ""
-      def packaged(s: String) = if (pkg.isEmpty) s else s"$pkg.$s"
+      // reconstitute an anonfun with a package
+      // if filtered, add the hash back, e.g. pkg.Foo#bar, pkg.Foo$anon$1#apply
+      def packaged(s: String) = {
+        val p = if (pkg.isEmpty) s else s"$pkg.$s"
+        val pm = filter map (p + "#" + _)
+        pm getOrElse p
+      }
       // is this translated path in (usually virtual) repl outdir? or loadable from filesystem?
       val fs = if (isReplish) {
         def outed(d: AbstractFile, p: Seq[String]): Option[AbstractFile] = {
@@ -628,7 +663,7 @@ object JavapClass {
       }
       fs match {
         case Some(xs) => xs.to[Seq]     // maybe empty
-        case None     => Seq(k0)        // just bail on fail
+        case None     => Seq()          // nothing found, e.g., junk input
       }
     }
     def funs(ks: Seq[String]) = ks flatMap funsOf _
diff --git a/test/files/run/repl-javap-outdir-funs/foo_1.scala b/test/files/run/repl-javap-outdir-funs/foo_1.scala
new file mode 100644
index 0000000000..9b98e94733
--- /dev/null
+++ b/test/files/run/repl-javap-outdir-funs/foo_1.scala
@@ -0,0 +1,6 @@
+
+package disktest
+
+class Foo {
+  def m(vs: List[Int]) = vs map (_ + 1)
+}
diff --git a/test/files/run/repl-javap-outdir-funs/run-repl_7.scala b/test/files/run/repl-javap-outdir-funs/run-repl_7.scala
new file mode 100644
index 0000000000..dfe3dae270
--- /dev/null
+++ b/test/files/run/repl-javap-outdir-funs/run-repl_7.scala
@@ -0,0 +1,12 @@
+import scala.tools.partest.JavapTest
+
+object Test extends JavapTest {
+  def code = """
+    |:javap -fun disktest/Foo.class
+  """.stripMargin
+
+  override def yah(res: Seq[String]) = {
+    def filtered = res filter (_ contains "public final class disktest.Foo")
+    1 == filtered.size
+  }
+}
-- 
cgit v1.2.3


From f7490d51bf03ca45749b767b0bf36131a466b2ee Mon Sep 17 00:00:00 2001
From: Som Snytt <som.snytt@gmail.com>
Date: Fri, 11 Jan 2013 11:38:14 -0800
Subject: Restore pending repl-javap tests that now succeed under java 6.

Knock on wood.
---
 test/files/run/repl-javap-def.scala                 | 17 +++++++++++++++++
 test/files/run/repl-javap-fun.scala                 | 16 ++++++++++++++++
 test/files/run/repl-javap-mem.scala                 | 19 +++++++++++++++++++
 test/files/run/repl-javap-memfun.scala              | 18 ++++++++++++++++++
 test/files/run/repl-javap-more-fun.scala            | 17 +++++++++++++++++
 test/files/run/repl-javap-outdir/foo_1.scala        |  6 ++++++
 test/files/run/repl-javap-outdir/run-repl_7.scala   | 12 ++++++++++++
 test/files/run/repl-javap.scala                     | 13 +++++++++++++
 test/pending/run/repl-javap-def.scala               | 17 -----------------
 test/pending/run/repl-javap-fun.scala               | 16 ----------------
 test/pending/run/repl-javap-mem.scala               | 19 -------------------
 test/pending/run/repl-javap-memfun.scala            | 18 ------------------
 test/pending/run/repl-javap-more-fun.scala          | 17 -----------------
 test/pending/run/repl-javap-outdir/foo_1.scala      |  6 ------
 test/pending/run/repl-javap-outdir/run-repl_7.scala | 12 ------------
 test/pending/run/repl-javap.scala                   | 13 -------------
 16 files changed, 118 insertions(+), 118 deletions(-)
 create mode 100644 test/files/run/repl-javap-def.scala
 create mode 100644 test/files/run/repl-javap-fun.scala
 create mode 100644 test/files/run/repl-javap-mem.scala
 create mode 100644 test/files/run/repl-javap-memfun.scala
 create mode 100644 test/files/run/repl-javap-more-fun.scala
 create mode 100644 test/files/run/repl-javap-outdir/foo_1.scala
 create mode 100644 test/files/run/repl-javap-outdir/run-repl_7.scala
 create mode 100644 test/files/run/repl-javap.scala
 delete mode 100644 test/pending/run/repl-javap-def.scala
 delete mode 100644 test/pending/run/repl-javap-fun.scala
 delete mode 100644 test/pending/run/repl-javap-mem.scala
 delete mode 100644 test/pending/run/repl-javap-memfun.scala
 delete mode 100644 test/pending/run/repl-javap-more-fun.scala
 delete mode 100644 test/pending/run/repl-javap-outdir/foo_1.scala
 delete mode 100644 test/pending/run/repl-javap-outdir/run-repl_7.scala
 delete mode 100644 test/pending/run/repl-javap.scala

diff --git a/test/files/run/repl-javap-def.scala b/test/files/run/repl-javap-def.scala
new file mode 100644
index 0000000000..dbd769613a
--- /dev/null
+++ b/test/files/run/repl-javap-def.scala
@@ -0,0 +1,17 @@
+import scala.tools.partest.JavapTest
+
+object Test extends JavapTest {
+  def code = """
+    |def f = 7
+    |:javap -public -raw f
+  """.stripMargin
+
+  // it should find f wrapped in repl skins. replstiltskin.
+  override def yah(res: Seq[String]) = {
+    // replstiltskin: what be my name?
+    val keywords = List("public", "class", "line")
+    def isLineClass(s: String) = keywords forall (s contains _)
+    def filtered = res filter isLineClass
+    1 == filtered.size
+  }
+}
diff --git a/test/files/run/repl-javap-fun.scala b/test/files/run/repl-javap-fun.scala
new file mode 100644
index 0000000000..5c9a6b7691
--- /dev/null
+++ b/test/files/run/repl-javap-fun.scala
@@ -0,0 +1,16 @@
+import scala.tools.partest.JavapTest
+
+object Test extends JavapTest {
+  def code = """
+    |object Betty {
+    | List(1,2,3) filter (_ % 2 != 0) map (_ * 2)
+    |}
+    |:javap -fun Betty
+  """.stripMargin
+
+  // two anonfuns of Betty
+  override def yah(res: Seq[String]) = {
+    def filtered = res filter (_ contains "public final class Betty")
+    2 == filtered.size
+  }
+}
diff --git a/test/files/run/repl-javap-mem.scala b/test/files/run/repl-javap-mem.scala
new file mode 100644
index 0000000000..8db30e835c
--- /dev/null
+++ b/test/files/run/repl-javap-mem.scala
@@ -0,0 +1,19 @@
+import scala.tools.partest.JavapTest
+
+object Test extends JavapTest {
+  def code = """
+    |object Betty {
+    |  val ds = List(1,2,3) filter (_ % 2 == 0) map (_ * 3)
+    |  def m(vs: List[Int]) = vs filter (_ % 2 != 0) map (_ * 2)
+    |}
+    |:javap Betty#m
+  """.stripMargin
+
+  // filter for requested method member
+  override def yah(res: Seq[String]) = {
+    // cheaply, methods end in arg list
+    val p = """.*m\(.*\);""".r
+    def filtered = res filter (_ match { case p() => true case _ => false })
+    1 == filtered.size
+  }
+}
diff --git a/test/files/run/repl-javap-memfun.scala b/test/files/run/repl-javap-memfun.scala
new file mode 100644
index 0000000000..d2b4243c8b
--- /dev/null
+++ b/test/files/run/repl-javap-memfun.scala
@@ -0,0 +1,18 @@
+import scala.tools.partest.JavapTest
+
+object Test extends JavapTest {
+  def code = """
+    |object Betty {
+    | List(1,2,3) count (_ % 2 != 0)
+    | def f = List(1,2,3) filter (_ % 2 != 0) map (_ * 2)
+    | def g = List(1,2,3) filter (_ % 2 == 0) map (_ * 3) map (_ + 1)
+    |}
+    |:javap -fun Betty#g
+  """.stripMargin
+
+  // three anonfuns of Betty#g
+  override def yah(res: Seq[String]) = {
+    def filtered = res filter (_ contains "public final class Betty")
+    3 == filtered.size
+  }
+}
diff --git a/test/files/run/repl-javap-more-fun.scala b/test/files/run/repl-javap-more-fun.scala
new file mode 100644
index 0000000000..e603faf75a
--- /dev/null
+++ b/test/files/run/repl-javap-more-fun.scala
@@ -0,0 +1,17 @@
+import scala.tools.partest.JavapTest
+
+object Test extends JavapTest {
+  def code = """
+    |object Betty {
+    |  val ds = List(1,2,3) filter (_ % 2 == 0) map (_ * 3)
+    |  def m(vs: List[Int]) = vs filter (_ % 2 != 0) map (_ * 2)
+    |}
+    |:javap -fun Betty
+  """.stripMargin
+
+  // two anonfuns of Betty
+  override def yah(res: Seq[String]) = {
+    def filtered = res filter (_ contains "public final class Betty")
+    4 == filtered.size
+  }
+}
diff --git a/test/files/run/repl-javap-outdir/foo_1.scala b/test/files/run/repl-javap-outdir/foo_1.scala
new file mode 100644
index 0000000000..9b98e94733
--- /dev/null
+++ b/test/files/run/repl-javap-outdir/foo_1.scala
@@ -0,0 +1,6 @@
+
+package disktest
+
+class Foo {
+  def m(vs: List[Int]) = vs map (_ + 1)
+}
diff --git a/test/files/run/repl-javap-outdir/run-repl_7.scala b/test/files/run/repl-javap-outdir/run-repl_7.scala
new file mode 100644
index 0000000000..dc2c5719ff
--- /dev/null
+++ b/test/files/run/repl-javap-outdir/run-repl_7.scala
@@ -0,0 +1,12 @@
+import scala.tools.partest.JavapTest
+
+object Test extends JavapTest {
+  def code = """
+    |:javap disktest/Foo.class
+  """.stripMargin
+
+  override def yah(res: Seq[String]) = {
+    def filtered = res filter (_ contains "public class disktest.Foo")
+    1 == filtered.size
+  }
+}
diff --git a/test/files/run/repl-javap.scala b/test/files/run/repl-javap.scala
new file mode 100644
index 0000000000..7a19852d4e
--- /dev/null
+++ b/test/files/run/repl-javap.scala
@@ -0,0 +1,13 @@
+import scala.tools.partest.JavapTest
+
+object Test extends JavapTest {
+  def code = """
+    |case class Betty(i: Int) { def next = Betty(i+1) }
+    |:javap Betty
+  """.stripMargin
+
+  override def yah(res: Seq[String]) = {
+    def filtered = res filter (_ contains "public class Betty")
+    1 == filtered.size
+  }
+}
diff --git a/test/pending/run/repl-javap-def.scala b/test/pending/run/repl-javap-def.scala
deleted file mode 100644
index dbd769613a..0000000000
--- a/test/pending/run/repl-javap-def.scala
+++ /dev/null
@@ -1,17 +0,0 @@
-import scala.tools.partest.JavapTest
-
-object Test extends JavapTest {
-  def code = """
-    |def f = 7
-    |:javap -public -raw f
-  """.stripMargin
-
-  // it should find f wrapped in repl skins. replstiltskin.
-  override def yah(res: Seq[String]) = {
-    // replstiltskin: what be my name?
-    val keywords = List("public", "class", "line")
-    def isLineClass(s: String) = keywords forall (s contains _)
-    def filtered = res filter isLineClass
-    1 == filtered.size
-  }
-}
diff --git a/test/pending/run/repl-javap-fun.scala b/test/pending/run/repl-javap-fun.scala
deleted file mode 100644
index 5c9a6b7691..0000000000
--- a/test/pending/run/repl-javap-fun.scala
+++ /dev/null
@@ -1,16 +0,0 @@
-import scala.tools.partest.JavapTest
-
-object Test extends JavapTest {
-  def code = """
-    |object Betty {
-    | List(1,2,3) filter (_ % 2 != 0) map (_ * 2)
-    |}
-    |:javap -fun Betty
-  """.stripMargin
-
-  // two anonfuns of Betty
-  override def yah(res: Seq[String]) = {
-    def filtered = res filter (_ contains "public final class Betty")
-    2 == filtered.size
-  }
-}
diff --git a/test/pending/run/repl-javap-mem.scala b/test/pending/run/repl-javap-mem.scala
deleted file mode 100644
index 8db30e835c..0000000000
--- a/test/pending/run/repl-javap-mem.scala
+++ /dev/null
@@ -1,19 +0,0 @@
-import scala.tools.partest.JavapTest
-
-object Test extends JavapTest {
-  def code = """
-    |object Betty {
-    |  val ds = List(1,2,3) filter (_ % 2 == 0) map (_ * 3)
-    |  def m(vs: List[Int]) = vs filter (_ % 2 != 0) map (_ * 2)
-    |}
-    |:javap Betty#m
-  """.stripMargin
-
-  // filter for requested method member
-  override def yah(res: Seq[String]) = {
-    // cheaply, methods end in arg list
-    val p = """.*m\(.*\);""".r
-    def filtered = res filter (_ match { case p() => true case _ => false })
-    1 == filtered.size
-  }
-}
diff --git a/test/pending/run/repl-javap-memfun.scala b/test/pending/run/repl-javap-memfun.scala
deleted file mode 100644
index d2b4243c8b..0000000000
--- a/test/pending/run/repl-javap-memfun.scala
+++ /dev/null
@@ -1,18 +0,0 @@
-import scala.tools.partest.JavapTest
-
-object Test extends JavapTest {
-  def code = """
-    |object Betty {
-    | List(1,2,3) count (_ % 2 != 0)
-    | def f = List(1,2,3) filter (_ % 2 != 0) map (_ * 2)
-    | def g = List(1,2,3) filter (_ % 2 == 0) map (_ * 3) map (_ + 1)
-    |}
-    |:javap -fun Betty#g
-  """.stripMargin
-
-  // three anonfuns of Betty#g
-  override def yah(res: Seq[String]) = {
-    def filtered = res filter (_ contains "public final class Betty")
-    3 == filtered.size
-  }
-}
diff --git a/test/pending/run/repl-javap-more-fun.scala b/test/pending/run/repl-javap-more-fun.scala
deleted file mode 100644
index e603faf75a..0000000000
--- a/test/pending/run/repl-javap-more-fun.scala
+++ /dev/null
@@ -1,17 +0,0 @@
-import scala.tools.partest.JavapTest
-
-object Test extends JavapTest {
-  def code = """
-    |object Betty {
-    |  val ds = List(1,2,3) filter (_ % 2 == 0) map (_ * 3)
-    |  def m(vs: List[Int]) = vs filter (_ % 2 != 0) map (_ * 2)
-    |}
-    |:javap -fun Betty
-  """.stripMargin
-
-  // two anonfuns of Betty
-  override def yah(res: Seq[String]) = {
-    def filtered = res filter (_ contains "public final class Betty")
-    4 == filtered.size
-  }
-}
diff --git a/test/pending/run/repl-javap-outdir/foo_1.scala b/test/pending/run/repl-javap-outdir/foo_1.scala
deleted file mode 100644
index 9b98e94733..0000000000
--- a/test/pending/run/repl-javap-outdir/foo_1.scala
+++ /dev/null
@@ -1,6 +0,0 @@
-
-package disktest
-
-class Foo {
-  def m(vs: List[Int]) = vs map (_ + 1)
-}
diff --git a/test/pending/run/repl-javap-outdir/run-repl_7.scala b/test/pending/run/repl-javap-outdir/run-repl_7.scala
deleted file mode 100644
index dc2c5719ff..0000000000
--- a/test/pending/run/repl-javap-outdir/run-repl_7.scala
+++ /dev/null
@@ -1,12 +0,0 @@
-import scala.tools.partest.JavapTest
-
-object Test extends JavapTest {
-  def code = """
-    |:javap disktest/Foo.class
-  """.stripMargin
-
-  override def yah(res: Seq[String]) = {
-    def filtered = res filter (_ contains "public class disktest.Foo")
-    1 == filtered.size
-  }
-}
diff --git a/test/pending/run/repl-javap.scala b/test/pending/run/repl-javap.scala
deleted file mode 100644
index 7a19852d4e..0000000000
--- a/test/pending/run/repl-javap.scala
+++ /dev/null
@@ -1,13 +0,0 @@
-import scala.tools.partest.JavapTest
-
-object Test extends JavapTest {
-  def code = """
-    |case class Betty(i: Int) { def next = Betty(i+1) }
-    |:javap Betty
-  """.stripMargin
-
-  override def yah(res: Seq[String]) = {
-    def filtered = res filter (_ contains "public class Betty")
-    1 == filtered.size
-  }
-}
-- 
cgit v1.2.3