Skip to content

replace: InterceptControllerをkotlinへ移行#234

Merged
funa-tk merged 1 commit intoDeNA:masterfrom
mayo031042:feature/replace-InterceptController
Mar 12, 2026
Merged

replace: InterceptControllerをkotlinへ移行#234
funa-tk merged 1 commit intoDeNA:masterfrom
mayo031042:feature/replace-InterceptController

Conversation

@mayo031042
Copy link

InterceptController.javaをkotlinへ置換しました

同時に以下の変更を加えています
参考:Arrow

  • センチネル値の利用を廃止
    • パケットdrop時の結果の表現を「空のバイト配列」から「Arrow ライブラリの Option 型」に変更
    • 将来的にパケットの処理を関数型パイプラインで記述しなおすことを目指した変更でもあります
  • DuplexFactory.javaから呼び出すためのwrapper関数を追加
    • Java コードから suspend 関数を呼び出すには Continuation を直接扱う必要があり実用的でないため、DuplexFactory.java から呼び出していた箇所は一旦 runBlocking でラップし、呼び出し元スレッドをブロックする互換ラッパー(receivedBlocking)を経由するようにしています
    • 将来的に DuplexFactory.java が Kotlin 化された際に削除予定です

@funa-tk
Copy link
Contributor

funa-tk commented Mar 12, 2026

特に問題ないと思いましたのでマージさせていただきます!

@funa-tk funa-tk merged commit c46c6b0 into DeNA:master Mar 12, 2026
1 check passed
@mayo031042 mayo031042 deleted the feature/replace-InterceptController branch March 12, 2026 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants