Skip to content

wcscpy 置換 (CControlTray)#2397

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

wcscpy 置換 (CControlTray)#2397
gorogoro123 wants to merge 1 commit intosakura-editor:masterfrom
gorogoro123:feature/replace_wcscpy_CControlTray

Conversation

@gorogoro123
Copy link
Copy Markdown
Contributor

@gorogoro123 gorogoro123 commented Feb 28, 2026

PR対象

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

カテゴリ

  • 改善

PR の背景

wcscpy を使用しているため、 build 時に warning が表示される。

warning C4996: 'wcscpy': This function or variable may be unsafe. Consider using wcscpy_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.

仕様・動作説明

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

PR の影響範囲

影響なし。

テスト内容

  1. デバッグ - コマンド に -NOWIN を設定する。
  2. タスクトレイから Grep を実行する。
  3. タスクトレイから 新規作成 - 設定 - 共通設定 - OK を選択する。

関連 issue, PR

参考資料

https://learn.microsoft.com/en-us/cpp/c-runtime-library/reference/strcpy-wcscpy-mbscpy

@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 89d9739. ± 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