From 4d54d1a8dc97b2b6ecbf2a5487e02b27a2b75f56 Mon Sep 17 00:00:00 2001 From: Katelyn Baker Date: Wed, 16 Aug 2017 11:39:07 +0100 Subject: [PATCH] Review Comments Removing unused default method on the factory, it was added for testing although given there are easier ways to build factories for the tests it was never used and doesn't need leaving in --- .../internal/serialization/AMQPSerializationScheme.kt | 7 ------- 1 file changed, 7 deletions(-) diff --git a/node-api/src/main/kotlin/net/corda/nodeapi/internal/serialization/AMQPSerializationScheme.kt b/node-api/src/main/kotlin/net/corda/nodeapi/internal/serialization/AMQPSerializationScheme.kt index e0a54e8a8a..eee7ed73c6 100644 --- a/node-api/src/main/kotlin/net/corda/nodeapi/internal/serialization/AMQPSerializationScheme.kt +++ b/node-api/src/main/kotlin/net/corda/nodeapi/internal/serialization/AMQPSerializationScheme.kt @@ -46,13 +46,6 @@ abstract class AbstractAMQPSerializationScheme : SerializationScheme { }.also { registerCustomSerializers(it) } } - fun getSerializerFactory(): SerializerFactory { - return serializerFactoriesForContexts.computeIfAbsent(Pair( - AllWhitelist, SerializationDefaults.javaClass.classLoader)) { - SerializerFactory(AllWhitelist, SerializationDefaults.javaClass.classLoader) - } - } - override fun deserialize(byteSequence: ByteSequence, clazz: Class, context: SerializationContext): T { val serializerFactory = getSerializerFactory(context) return DeserializationInput(serializerFactory).deserialize(byteSequence, clazz)