Fixed #71: OOM cause by tracing method with huge-object argument.

CSUG · Sep 1, 2012 · 1fceb3c · 1fceb3c
1 parent 7b1dbfe
commit 1fceb3c
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/main/scala/com/github/zhongl/housemd/instrument/Context.scala b/src/main/scala/com/github/zhongl/housemd/instrument/Context.scala
@@ -26,7 +26,7 @@ case class Context(
   className: String,
   methodName: String,
   loader: ClassLoader,
-  arguments: Array[AnyRef],
+  arguments: Array[String],
   descriptor: String,
   isVoidReturn: Boolean,
   thisObject: AnyRef,
@@ -38,15 +38,13 @@ case class Context(
 
 object Context {
 
-  import java.util.Map
+  implicit val map2Context:(java.util.Map[String,AnyRef] => Context) = apply(_: java.util.Map[String, AnyRef])
 
-  implicit val map2Context = apply(_: Map[String, AnyRef])
-
-  def apply(map: util.Map[String, AnyRef]) = new Context(
+  def apply(map: java.util.Map[String, AnyRef]) = new Context(
     map.get(Advice.CLASS).asInstanceOf[String],
     map.get(Advice.METHOD).asInstanceOf[String],
     map.get(Advice.CLASS_LOADER).asInstanceOf[ClassLoader],
-    map.get(Advice.ARGUMENTS).asInstanceOf[Array[AnyRef]],
+    map.get(Advice.ARGUMENTS).asInstanceOf[Array[AnyRef]].map(_.toString),
     map.get(Advice.DESCRIPTOR).asInstanceOf[String],
     map.get(Advice.VOID_RETURN).asInstanceOf[Boolean],
     map.get(Advice.THIS),