-
Notifications
You must be signed in to change notification settings - Fork 168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
指定桁縦線の指定できる数を増やす #1945
指定桁縦線の指定できる数を増やす #1945
Conversation
✅ Build sakura 1.0.4330 completed (commit 7409016b8f by @kurages) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR ありがとうございます。制限を緩和することによるデメリットが思い浮かばないのでマージ可能だと思うのですが、その前に一点だけ指摘事項があります…!(MinGW ビルドがこけているのは明らかに別件なので考えません)
sakura/sakura_core/config/system_constants.h
Lines 553 to 561 in 0345a2f
Version 178: | |
STypeConfig::m_bDetectIndentationStyleOnFileLoad 追加 | |
-- 統合されたので元に戻す(1000~1023が使用済み) 2008.11.16 nasukoji | |
-- Version 1000: | |
-- バージョン1000以降を本家統合までの間、使わせてください。かなり頻繁に構成が変更されると思われるので。by kobake 2008.03.02 | |
*/ | |
#define N_SHAREDATA_VERSION 178 |
N_SHAREDATA_VERSION
をインクリメントしてください。理由を説明するコメントの記入もお願いします。
STypeConfig::m_nVertLineIdx
が MAX_VERTLINES
を要素数とする配列であるので、MAX_VERTLINES
の値を増やすと STypeConfig
型の変数をメンバーに持つ DLLSHAREDATA
(プロセス間共有メモリ構造体)のメモリレイアウトが変わります。異なるバージョンのエディタを同時に実行した場合の衝突を回避する目的でバージョン付けを行っているので、ここの値の更新が必要になります。
レニューありがとうございます! |
不要な差分が発生していたので戻しました |
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
✅ Build sakura 1.0.4331 completed (commit 808570253d by @kurages) |
✅ Build sakura 1.0.4332 completed (commit e32adc8df7 by @kurages) |
✅ Build sakura 1.0.4333 completed (commit cceb31bcec by @kurages) |
問題があれば適宜 revert する前提でマージしてみました。ありがとうございました。 |
PR の目的
指定桁縦線の指定できる数を増やす
PR対象
カテゴリ
PR の背景
システム間連携の固定長フォーマットファイル(全銀協フォーマットの様なものです)を開く際に
項目の区切り位置に指定桁縦線を設定しています。
項目数が10を超えるフォーマットもたくさんあるので、指定できる線の数を増やしたい要望になります。
使用しているファイルの中で項目数が多いフォーマットで170前後のため、
とりあえず200まで増やしています。
イメージ
仕様・動作説明
指定桁縦線の指定可能な数を10から200に増やします
PR の影響範囲
テスト内容
GitHub Actionsでビルドされた成果物(Release Win32, Release x64)で動作の確認はしました。
関連 issue, PR
参考資料