Skip to content

コード改善#374

Merged
sayurin merged 5 commits intomasterfrom
reduce-warning
May 4, 2022
Merged

コード改善#374
sayurin merged 5 commits intomasterfrom
reduce-warning

Conversation

@sayurin
Copy link
Copy Markdown
Member

@sayurin sayurin commented May 4, 2022

コード分析で警告される部分について、改善し、コード分析を有効化する。
指摘は多岐にわたるのでちょっとずつ…。またコンパイラー側のバグで誤検出もありそういうのは無視する。

@sayurin sayurin self-assigned this May 4, 2022
@sayurin
Copy link
Copy Markdown
Member Author

sayurin commented May 4, 2022

警告とは別に std::views::iota() を使うことで for ループを 範囲ベースのfor文 に置き換えることができる。これによりループ変数を const 化できるようになる。

@sayurin sayurin added this to the v5.7 milestone May 4, 2022
@sayurin sayurin merged commit 40e235d into master May 4, 2022
@sayurin sayurin deleted the reduce-warning branch May 4, 2022 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant