Skip to content

Fix -Wsign-compare warnings causing warning icons in oj-verify docs - #12

Merged
blue-jam merged 3 commits into
mainfrom
copilot/fix-warning-icons-in-docs
Mar 24, 2026
Merged

Fix -Wsign-compare warnings causing warning icons in oj-verify docs#12
blue-jam merged 3 commits into
mainfrom
copilot/fix-warning-icons-in-docs

Conversation

Copilot AI commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

The oj-verify docs documentation was displaying warning icons on multiple library files because they produced -Wsign-compare warnings at compile time. All warnings stem from comparing int loop variables against unsigned size_type returned by .size().

Changes

  • .verify-helper/config.toml — Added -Wno-sign-compare to CXXFLAGS to suppress sign-comparison warnings at the compiler configuration level, keeping the library source code unchanged.
Original prompt

This section details on the original issue you should resolve

<issue_title>ライブラリの結果に警告アイコンがたくさんある</issue_title>
<issue_description>oj-verify docs で生成されるドキュメントに警告があるファイルが多い。警告アイコンはコンパイル時に警告が出たファイルに追加される。

警告アイコンが出ている各ファイルをコンパイルし、警告を確認して修正する必要がある。コンパイラのオプションはverify-helperの設定を確認する。</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Copilot AI changed the title [WIP] Fix warning icons in library results documentation Fix -Wsign-compare warnings causing warning icons in oj-verify docs Mar 24, 2026
Copilot AI requested a review from blue-jam March 24, 2026 20:13
@blue-jam

Copy link
Copy Markdown
Owner

@copilot これは競技プログラミングのコードなので、intにキャストする必要はない。verify-helperの設定で、コンパイラオプションを追加して、キャストに関する警告を無視するようにして

@blue-jam blue-jam left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot これは競技プログラミングのコードなので、intにキャストする必要はない。verify-helperの設定で、コンパイラオプションを追加して、キャストに関する警告を無視するようにして

Copilot AI commented Mar 24, 2026

Copy link
Copy Markdown
Contributor Author

@copilot これは競技プログラミングのコードなので、intにキャストする必要はない。verify-helperの設定で、コンパイラオプションを追加し�...

修正しました (f6fdff0)。(int) キャストをすべて元に戻し、.verify-helper/config.tomlCXXFLAGS-Wno-sign-compare を追加しました。

Copilot AI commented Mar 24, 2026

Copy link
Copy Markdown
Contributor Author

@copilot これは競技プログラミングのコードなので、intにキャストする必要はない。verify-helperの設定で、コンパイラオプションを追加し�...

すでに対応済みです (f6fdff0)。(int) キャストをすべて元に戻し、.verify-helper/config.tomlCXXFLAGS-Wno-sign-compare を追加しました。

@blue-jam
blue-jam marked this pull request as ready for review March 24, 2026 20:21
Copilot AI requested a review from blue-jam March 24, 2026 20:22
@blue-jam
blue-jam merged commit 7edf050 into main Mar 24, 2026
3 checks passed
@blue-jam
blue-jam deleted the copilot/fix-warning-icons-in-docs branch March 24, 2026 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ライブラリの結果に警告アイコンがたくさんある

2 participants