Skip to content

エディター初期化完了イベントを導入する#2532

Merged
berryzplus merged 19 commits into
sakura-editor:masterfrom
berryzplus:feature/add_editor_initialized
Jul 18, 2026
Merged

エディター初期化完了イベントを導入する#2532
berryzplus merged 19 commits into
sakura-editor:masterfrom
berryzplus:feature/add_editor_initialized

Conversation

@berryzplus

Copy link
Copy Markdown
Contributor

PR対象

  • アプリ(サクラエディタ本体)
  • テストコード

カテゴリ

  • 改善

PR の背景

サクラエディタには、エディターの初期化完了を検出する方法がありません。

このため、編集ウインドウの作成、ファイルを開く、Grep実行などのテストを書くことができません。

※厳密には、書くこと自体はできるが、初期化完了を検知できないのでちょくちょく失敗する。

仕様・動作説明

  • コントロールプロセス側の初期化完了イベントを参考に、エディター初期化完了イベントを追加します。
  • アプリ側変更を最小限にし、テストで使えるかどうか検証するための仮仕様として実装します。

本件、勝手にやります。

PR の影響範囲

  • この変更により、Grep実行のテストで余分な完了待ちをする必要がなくなります。

テスト内容

関連 issue, PR

参考資料

CMutexの親クラスにできるよう、lockのconstを外す。
表示されたダイアログのクローズに実績のあるやり方を使う。
CProcessFactoryの固有機能ではないため、いずれ他クラスに移動する。
実際の移動まで行うと差分が大きくなりすぎるので、一旦static化のみに留める。
WCHAR[260]同士のコピーなので実は「安全」と断言できるのだけど、面倒なので直してしまう。
@github-actions

Copy link
Copy Markdown

Test Results

1 116 tests  +5   1 116 ✅ +5   3m 35s ⏱️ + 1m 3s
  106 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit e0e0fe7. ± Comparison against base commit 654e67c.

@sonarqubecloud

Copy link
Copy Markdown

@berryzplus
berryzplus marked this pull request as ready for review July 18, 2026 00:05
@berryzplus
berryzplus merged commit 5d2d293 into sakura-editor:master Jul 18, 2026
17 checks passed
@berryzplus
berryzplus deleted the feature/add_editor_initialized branch July 18, 2026 00:06
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.

エディターの初期化完了を検知する手段を用意したい

1 participant