SonarScanで検出されたBugsレベル警告に対処する #1554
Merged
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
|
Kudos, SonarCloud Quality Gate passed! |
|
✅ Build sakura 1.0.3481 completed (commit 734ec8e5a1 by @berryzplus) |
kengoide
approved these changes
Feb 26, 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 の目的
SonarScanで検出されたBugsレベルの警告に対策して、より信頼できるエディタに近づけます。
カテゴリ
PR の背景
SonarScanで以下のBugsレベル警告が検出されています。
PR のメリット
Bugsレベル警告を1個潰せます。
PR のデメリット (トレードオフとかあれば)
とくにないと思います。
仕様・動作説明
レイアウト単位をロジック単位に変換する処理のコードを修正します。
レイアウト単位＝行番号、桁位置
👇
拡張ロジック単位＝行番号、行頭からのオフセット位置＋レイアウト情報
👇
ロジック単位＝行番号、行頭からのオフセット位置
内部的に行っている2個目の変換について警告が出ています。
対策としては、コピー代入をムーブ代入に置き換えています。
PR の影響範囲
レイアウト単位をロジック単位に変換する処理に影響します。
テスト内容
既に実施している単体テストでカバーされる範囲の修正なので、追加テストは不要と考えます。
関連 issue, PR
#1504
参考資料