sakura
sakura copied to clipboard
SAKURA Editor (Japanese text editor for MS Windows)
トップディレクトリのファイル数が多くなってきたので、整理する https://github.com/sakura-editor/sakura/pull/826#issuecomment-479873527 > > もっと理想を言えば、appveyor と共通で利用するバッチ類を `ci` ディレクトリに、appveyor のものを `ci/appveyor` ディレクトリに > > まあ、ぼちぼちと考えます。
## 発端 - #1402 を再検討したい。 ## 経緯 - Visual Studio 2026がリリースされた。 - サポートするVCビルド環境が4つになってしまった。 - C++20を採用できなかった理由「vs2017が対応してない」を考慮しなくて良くなるかもしれない。 - 文字列バッファに std::span を使う環境では std::size に寄せるメリットが出てくる。 - 独自マクロ _countof2 を消したい。 レベル感違う列挙が混じってるのはスルーで。
# PR対象 - アプリ(サクラエディタ本体) ## カテゴリ - 改善 ## PR の背景 - #2191 共有データに依存するコードのテストを追加しました。 - #2200 共有データに依存するコードの本命、CEditWndをテストできるようにしました。 CEditWndの改善に進みたかったんですが、 先に DLLSHAREDATA の問題に対処すべきと判断してPRを分けます。 レビューつかないと思うので、本件勝手にやります。 ## 仕様・動作説明 - 共有データは `GetDllShareData()` で取得できるグローバルオブジェクトです。 - 共有データの実体は...
# PR対象 - アプリ(サクラエディタ本体) ## カテゴリ - 改善 ## PR の背景 - #2157 レビューコメント ## 仕様・動作説明 - in-class initializeの追加対応(m_uCreateTaskBarMsg) - 初期化できないメンバー m_szLanguageDll の型を変更。(生配列→SFilePath) - m_uCreateTaskBarMsgをクラス定数に変更。 - リネーム m_uCreateTaskBarMsg → gm_uMsgTaskbarCreated。...
トップディレクトリのファイルが多くなってしまったことの対策。 - 妥当な移動先を検討して移動したい。 - 移動が完了したら閉じる。 - 検討した結果「移動しない」になったらその旨記載して閉じる。 - 移動に先立つ課題があればここにぶら下げて、課題解決後に続行する。
## 問題内容 ``` 審査を通るだけの安全性・妥当性を示せること(未達成) → 非推奨 API(マルチスレッドアプリでは使用すべきでないものなど)が含まれており、テストカバレッジも 30% を下回っているため、現状では審査を通過する可能性が低いです。 ``` (#2145 のコメント抜粋。) ## 再現手順 - CIに含まれる SonarQube Scan 解析結果を確認。(30%を下回っている。) https://sonarcloud.io/summary/overall?id=sakura-editor_sakura&branch=master - セキュリティの観点から「絶対に使ってはいけない関数」を使用している。 https://github.com/sakura-editor/sakura/blob/9961b295757a2a26730f9a6ab187c802bf03f85e/sakura_core/macro/CPPA.cpp#L351 (「非推奨」というより「使用禁止」レベルで使っちゃダメなやつ。) ## 再現頻度 100% ## 問題のカテゴリ -...
トップディレクトリのファイルが多くなってしまったことの対策。 - 妥当な移動先を検討して移動したい。 - 移動が完了したら閉じる。 - 検討した結果「移動しない」になったらその旨記載して閉じる。 - 移動に先立つ課題があればここにぶら下げて、課題解決後に続行する。
トップディレクトリのファイルが多くなってしまったことの対策。 - 妥当な移動先を検討して移動したい。 - 移動が完了したら閉じる。 - 検討した結果「移動しない」になったらその旨記載して閉じる。 - 移動に先立つ課題があればここにぶら下げて、課題解決後に続行する。
トップディレクトリのファイルが多くなってしまったことの対策。 - 妥当な移動先を検討して移動したい。 - 移動が完了したら閉じる。 - 検討した結果「移動しない」になったらその旨記載して閉じる。 - 移動に先立つ課題があればここにぶら下げて、課題解決後に続行する。
トップディレクトリのファイルが多くなってしまったことの対策。 - 妥当な移動先を検討して移動したい。 - 移動が完了したら閉じる。 - 検討した結果「移動しない」になったらその旨記載して閉じる。 - 移動に先立つ課題があればここにぶら下げて、課題解決後に続行する。 Pull Request #2047 とも関連。