Merge pull request #290 from hveiga/move-clazz-instantiation-artery-d…

…eserialization Only instantiate clazz when includeManifest is enabled
altoo-ag · Jul 14, 2023 · 722489d · 722489d
2 parents c5b2ed2 + 2242b87
commit 722489d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...serialization/src/main/scala/io/altoo/akka/serialization/kryo/KryoSerializerBackend.scala b/...serialization/src/main/scala/io/altoo/akka/serialization/kryo/KryoSerializerBackend.scala
@@ -73,13 +73,13 @@ private[kryo] class KryoSerializerBackend(val kryo: Kryo, val bufferSize: Int, v
   // Implements ByteBufferSerializer
   override def fromBinary(buf: ByteBuffer, manifest: String): AnyRef = {
     val buffer = getInput(buf)
-    val clazz = system.dynamicAccess.getClassFor[AnyRef](manifest)
-    if (includeManifest)
+    if (includeManifest) {
+      val clazz = system.dynamicAccess.getClassFor[AnyRef](manifest)
       clazz match {
         case Success(c) => kryo.readObject(buffer, c).asInstanceOf[AnyRef]
         case _ => throw new RuntimeException("Object of unknown class cannot be deserialized")
       }
-    else
+    } else
       kryo.readClassAndObject(buffer)
   }