指定桁縦線の指定できる数を増やす #1945
指定桁縦線の指定できる数を増やす #1945
Conversation
|
✅ Build sakura 1.0.4330 completed (commit 7409016b8f by @kurages) |
There was a problem hiding this comment.
PR ありがとうございます。制限を緩和することによるデメリットが思い浮かばないのでマージ可能だと思うのですが、その前に一点だけ指摘事項があります…！（MinGW ビルドがこけているのは明らかに別件なので考えません）
sakura/sakura_core/config/system_constants.h
Lines 553 to 561 in 0345a2f
N_SHAREDATA_VERSION をインクリメントしてください。理由を説明するコメントの記入もお願いします。
STypeConfig::m_nVertLineIdx が MAX_VERTLINES を要素数とする配列であるので、MAX_VERTLINES の値を増やすと STypeConfig 型の変数をメンバーに持つ DLLSHAREDATA （プロセス間共有メモリ構造体）のメモリレイアウトが変わります。異なるバージョンのエディタを同時に実行した場合の衝突を回避する目的でバージョン付けを行っているので、ここの値の更新が必要になります。
|
レニューありがとうございます！ |
|
不要な差分が発生していたので戻しました |
|
|
✅ 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
参考資料