Skip to content

Grep機能拡張(マルチスレッド、除外ファイルの正規表現)#2449

Open
hpmy-dev wants to merge 3 commits intosakura-editor:masterfrom
hpmy-dev:feature/grepmatulithreads
Open

Grep機能拡張(マルチスレッド、除外ファイルの正規表現)#2449
hpmy-dev wants to merge 3 commits intosakura-editor:masterfrom
hpmy-dev:feature/grepmatulithreads

Conversation

@hpmy-dev
Copy link
Copy Markdown

変更ファイル一覧

ビルドエラー修正

std::size(member)std::extent_v<decltype(member)> への置き換え(11箇所)。C++20準拠のMSVCで非静的メンバ変数への std::size() が定数式として評価できない問題を修正。

ファイル パス
CDlgCompare.cpp sakura_core/dlg/
CDlgDiff.cpp sakura_core/dlg/
CDlgFavorite.cpp sakura_core/dlg/
CDlgTagJumpList.cpp sakura_core/dlg/
CDlgWindowList.cpp sakura_core/dlg/
CDlgFuncList.cpp sakura_core/outline/
CShareData_IO.cpp sakura_core/env/
CDlgTypeList.cpp sakura_core/typeprop/
CType_Erlang.cpp sakura_core/types/

@hpmy-dev hpmy-dev changed the title Feature/grepmatulithreads Grep機能拡張(マルチスレッド、除外ファイルの正規表現) Apr 17, 2026
@beru
Copy link
Copy Markdown
Contributor

beru commented Apr 19, 2026

関連issue : #2439

@beru
Copy link
Copy Markdown
Contributor

beru commented Apr 19, 2026

@hpmy-dev 一部のファイルで改行コードが LF から CRLF に変わった関係で差分が確認しづらくなっています。

GitHub の Files changed タブの Open diff view settings で Hide whitespace のチェックを入れると改行コードの差異は表示されなくなりますが、改行コードの変更自体は不要だと思います。

なおWikiを見たんですが改行コードを何で統一する、という記載は見つける事が出来ませんでした。

@hpmy-dev
Copy link
Copy Markdown
Author

hpmy-dev commented Apr 19, 2026

mac環境でclone、ソース修正をadd,pushした為と思います。
Windows環境でやり直します。

@hpmy-dev hpmy-dev force-pushed the feature/grepmatulithreads branch from d988516 to 904494c Compare April 20, 2026 05:37
@github-actions
Copy link
Copy Markdown

Test Results

1 076 tests  ±0   1 076 ✅ ±0   6m 18s ⏱️ + 1m 8s
   99 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 904494c. ± Comparison against base commit 20c3efd.

@hpmy-dev
Copy link
Copy Markdown
Author

差分が確認できるかと思います。

https://github.com/sakura-editor/sakura/pull/2449/changes

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
5 Security Hotspots
4.5% Coverage on New Code (required ≥ 80%)
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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.

2 participants