dorkbox/Network
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

take/return kryo no longer need to suspend

Browse Source
This commit is contained in:
nathan 2020-08-18 22:37:08 +02:00
parent 7df974ce40
commit 345de4d734
2 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/dorkbox/network/serialization/NetworkSerializationManager.kt
View File

@ -80,12 +80,12 @@ interface NetworkSerializationManager : SerializationManager<DirectBuffer> {
/** /**
* @return takes a kryo instance from the pool. * @return takes a kryo instance from the pool.
*/ */
suspend fun takeKryo(): KryoExtra fun takeKryo(): KryoExtra
/** /**
* Returns a kryo instance to the pool. * Returns a kryo instance to the pool.
*/ */
suspend fun returnKryo(kryo: KryoExtra) fun returnKryo(kryo: KryoExtra)
/** /**
* @return true if the remote kryo registration are the same as our own * @return true if the remote kryo registration are the same as our own

7
src/dorkbox/network/serialization/Serialization.kt
View File

@ -555,7 +555,7 @@ class Serialization(private val references: Boolean,
/** /**
* @return takes a kryo instance from the pool. * @return takes a kryo instance from the pool.
*/ */
override suspend fun takeKryo(): KryoExtra { override fun takeKryo(): KryoExtra {
// ALWAYS get as many as needed. Recycle them to prevent too many getting created // ALWAYS get as many as needed. Recycle them to prevent too many getting created
return kryoPool.poll() ?: initKryo() return kryoPool.poll() ?: initKryo()
} }
@ -563,8 +563,9 @@ class Serialization(private val references: Boolean,
/** /**
* Returns a kryo instance to the pool for use later on * Returns a kryo instance to the pool for use later on
*/ */
override suspend fun returnKryo(kryo: KryoExtra) { override fun returnKryo(kryo: KryoExtra) {
kryoPool.send(kryo) // return as much as we can. don't suspend if the pool is full, we just throw it away.
kryoPool.offer(kryo)
} }
/** /**