個人開発 » beatoraja

実装に着手しました。

• ブランチ: feature/477
• 対応方針: libgdxネイティブリソース(Texture/Pixmap)の解放漏れ・解放後参照を修正する。

着手前調査での重要な修正(スコープ変更)¶

項目#1 FFmpegProcessor は誤診断のためスキップします。

• pixmap は new Pixmap(new Gdx2DPixmap(frame.image(), nativeData)) で nativeData[0]=0(basePtr=0)として生成されており、デコーダーの Arena.ofShared() が所有する rgbBuffer.asByteBuffer() へのビューにすぎません。ネイティブメモリは Pixmap ではなく Arena が所有し、grabber.close()(dispose時)で解放されます。
• libgdx 1.14.2 ソース確認済: Gdx2DPixmap(ByteBuffer, long[]) は basePtr=nativeData[0]=0、dispose() → free(0) → gdx2d_free((gdx2d_pixmap*)0) で NULL参照 → EXCEPTION_ACCESS_VIOLATION となります(FFmpegProcessor:176 の既存コメントが警告しているのはまさにこのクラッシュ)。
• restart() の pixmap=null は Java ラッパーを GC させるだけで、ネイティブリークはありません。現状コードは正しく、disposeを追加すべきではありません。
• したがって #1 はコード変更せず、本コメントを根拠記録とします。

実装対象 (#2〜#8)¶

• #2 SkinTimingVisualizer: 曲切替時に旧 backtex Texture を dispose してから再生成
• #3 PomyuCharaLoader: ダミー pixmap(1x1) を生成直後に dispose、未参照時の transparent Texture も解放
• #4 BGAProcessor: dispose() で blanktex を解放
• #5 MainController: 終了時 sprite/systemfont/messageRenderer/plainTexture/input を解放
• #6 SearchTextField: super.dispose() を再有効化し cursor/selection Texture も解放
• #7 ShaderManager: dispose() 後に shaders.clear()
• #8 SkinPlayDensityGraph: サイズ変更時に Texture も再生成して描画不整合を解消

テストについて¶

これらは GPU/ネイティブリソースのライフサイクル修正で、実 GL コンテキストが必須です。本リポジトリのテストハーネスには GL コンテキスト/Mockito が無く単体テスト不可のため、コンパイル検証 + チケット記載の手動メモリ監視(長時間プレイ)での検証とします。

対応が完了しました。

概要¶

libgdx の Disposable なネイティブリソース（Texture/Pixmap）の解放漏れ・解放後参照を修正し、プレイ/曲切替/終了のたびに蓄積していた JVM 外メモリのリークを解消した。

変更点¶

• SkinTimingVisualizer: BMSModel 変更時に旧 backtex Texture を解放してから再生成（曲切替ごとの Texture 蓄積を解消）。
• PomyuCharaLoader: ダミー pixmap(1x1) を生成直後に dispose。transparent Texture は未参照時のみ解放（transparentUsed で所有権を一意化し、未参照リークと double-dispose の両方を回避）。
• BGAProcessor: dispose() で blanktex(1x1黒) を解放（プレイ回数ぶんの蓄積を解消）。
• MainController: 終了時に input・sprite・systemfont・messageRenderer・共有 plain Texture を解放。
• SearchTextField: super.dispose() を復活させ Stage 内部リソースを解放。cursor/selection の Texture も解放。
• ShaderManager: dispose() 後に static shaders Map を clear()（解放後参照を防止）。
• SkinPlayDensityGraph: サイズ変更時に Texture も再生成し描画不整合を解消。metrics==null の早期 return を廃止。
• 変更ファイル: SkinTimingVisualizer.java、PomyuCharaLoader.java、BGAProcessor.java、MainController.java、SearchTextField.java、ShaderManager.java、SkinPlayDensityGraph.java の 7 件。

スコープ外（項目#1 / 意図的に未変更）¶

• FFmpegProcessor は誤診断のため変更なし。動画フレーム pixmap は basePtr=0 でデコーダーの Arena.ofShared() 所有バッファへのビューにすぎず、ネイティブリークは存在しない（grabber.close() で解放）。提案された pixmap.dispose() は libgdx 1.14.2 で gdx2d_free((gdx2d_pixmap*)0) の NULL 参照クラッシュ（FFmpegProcessor:176 の既存コメントが警告するもの）を招くため追加しない。

テスト¶

• .\gradlew.bat compileJava : 成功
• .\gradlew.bat test : 成功（BUILD SUCCESSFUL、回帰なし）
• 補足: GPU/ネイティブリソースのライフサイクル修正で実 GL コンテキストが必須。テストハーネスに GL コンテキスト/Mockito が無く単体テスト不可のため、コンパイル＋フルテスト回帰確認＋手動メモリ監視で検証。

関連 MR¶

• !152 (feature/477 → develop)