CMemory変更のデグレ対策 #1629
Merged
CMemory変更のデグレ対策 #1629
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
#1618変更前のコードでは、サイズゼロのインスタンスをコピーした場合にメモリが確保されていたのが、#1618の変更で確保されなくなってしまった。処理内容的には後者が正しいが、#1626の不具合が発生したので暫定的に元に戻す。
|
Kudos, SonarCloud Quality Gate passed! |
|
✅ Build sakura 1.0.3677 completed (commit 3382b81392 by @berryzplus) |
sanomari
added
🐛bug🦋
sanomari
approved these changes
Apr 17, 2021
|
レビューありがとうございます。マージしちゃいます。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
タイトル通りです。
カテゴリ
PR の背景
#1618 で行ったメモリバッファの変更で、既存の挙動を変更する修正を入れてしまっていました。
発生した不具合は #1626 で「SAKURA EDITORでGrep」が機能しなくなっています。
PR のメリット
#1626 が解消します。
PR のデメリット (トレードオフとかあれば)
とくにありません。
仕様・動作説明
コミットとソースのコメントを確認してください。
位置づけ的には、既存コードを壊さないために「不要な処理」をあえて入れておく、になります。
PR の影響範囲
#1618 と同じです。
テスト内容
VisualStudioのデバッグ設定で引数に以下を設定してデバッグ起動します。
修正前はstrGrepKey（Grep検索の検索キー）を設定しようとしてVAが発生します。
修正後は問題なく続行できるように戻ります。
関連 issue, PR
#1618
close #1626
close #1628
参考資料