kscience/controls-kt
8
0
Fork 0
Code Issues Pull Requests 2 Packages Projects Releases 1 Wiki Activity

Remove non-blocking mode for ports to avoid CPU clogging.

Browse Source
This commit is contained in:
Alexander Nozik 2023-05-27 22:44:30 +03:00
parent ced42779be
commit 22359a5570
2 changed files with 6 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
controls-core/src/commonMain/kotlin/space/kscience/controls/ports/Port.kt
View File

@ -18,10 +18,10 @@ public interface Port : ContextAware, Socket<ByteArray>
* A specialized factory for [Port]
*/
@Type(PortFactory.TYPE)
public interface PortFactory: Factory<Port>{
public interface PortFactory : Factory<Port> {
public val type: String
public companion object{
public companion object {
public const val TYPE: String = "controls.port"
}
}
@ -53,12 +53,10 @@ public abstract class AbstractPort(
/**
* Internal method to receive data synchronously
*/
protected fun receive(data: ByteArray) {
scope.launch {
protected suspend fun receive(data: ByteArray) {
logger.debug { "${this@AbstractPort} RECEIVED: ${data.decodeToString()}" }
incoming.send(data)
}
}
private val sendJob = scope.launch {
for (data in outgoing) {

5
controls-core/src/jvmMain/kotlin/space/kscience/controls/ports/ChannelPort.kt
View File

@ -87,9 +87,7 @@ public object TcpPort : PortFactory {
port: Int,
coroutineContext: CoroutineContext = context.coroutineContext,
): ChannelPort = ChannelPort(context,coroutineContext){
SocketChannel.open(InetSocketAddress(host, port)).apply {
configureBlocking(false)
}
SocketChannel.open(InetSocketAddress(host, port))
}
override fun build(context: Context, meta: Meta): ChannelPort {
@ -115,7 +113,6 @@ public object UdpPort : PortFactory {
): ChannelPort = ChannelPort(context,coroutineContext){
DatagramChannel.open().apply {
connect(InetSocketAddress(host, port))
configureBlocking(false)
}
}