Skip to content

wcsncpy 置換 (CNormalProcess::InitializeProcess)#2398

Open
gorogoro123 wants to merge 1 commit intosakura-editor:masterfrom
gorogoro123:feature/replace_wcsncpy_InitializeProcess
Open

wcsncpy 置換 (CNormalProcess::InitializeProcess)#2398
gorogoro123 wants to merge 1 commit intosakura-editor:masterfrom
gorogoro123:feature/replace_wcsncpy_InitializeProcess

Conversation

@gorogoro123
Copy link
Copy Markdown
Contributor

PR対象

  • アプリ(サクラエディタ本体)

カテゴリ

  • 改善

PR の背景

CNormalProcess::InitializeProcess で wcsncpy() 呼び出し、null 文字を設定している。

仕様・動作説明

SFilePathLong(StaticString) の operator (=) を使って文字列のコピーを行います。

PR の影響範囲

影響なし。

テスト内容

  1. デバッグ - コマンド引数に 下記を設定して確認する。
-GREPMODE -GREPDLG -GKEY="sakura" -GFILE="*.cpp;*.h;*.hpp" -GFOLDER="C:\work\sakura\sakura_core"
  1. 検索ファイル (GFILE)、検索フォルダ (GFOLDER) に長い文字列(512文字)を設定して確認する。

関連 issue, PR

#2153

参考資料

@github-actions
Copy link
Copy Markdown

Test Results

908 tests  ±0   908 ✅ ±0   2m 35s ⏱️ -1s
 86 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 65ef66b. ± Comparison against base commit b4f29b9.

@sonarqubecloud
Copy link
Copy Markdown

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