Harness header hygiene: trailing semicolons, name collisions, argument capture
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 50/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
調査の方向性
Start by inspecting the listed harness/tsffs-*.h and harness/tsffs.h macro definitions, then check harness/test.sh:25-26 and the MSVC C++ branch. Compare tsffs-gcc-x86.h with tsffs-gcc-x86_64.h and the riscv32/riscv64 headers; verify the reported issues with focused compile cases, including if (x) HARNESS_STOP(); else ... and an argument named value. Done means the listed header and script issues are corrected and the harness checks pass.
索引モデルが issue の本文から書いたものです。
説明
Where
- all
harness/tsffs-*.handharness/tsffs.h
Problem
- All macro bodies end in
} while (0);, breakingif (x) HARNESS_STOP(); else .... - x86 headers define
__cpuid(collides with<cpuid.h>/<intrin.h>); arm headers define__stringify(collides withlinux/stringify.h). - The local
valuein x86 macros captures a user argument namedvalue:HARNESS_START_WITH_MAXIMUM_SIZE(buf, value)passes the magic number. tsffs-gcc-x86.hduplicatestsffs-gcc-x86_64.h; riscv32/riscv64 differ only in whitespace.harness/test.sh:25-26missing/inrmpaths; MSVC C++ branch usessize_twith only<cstdint>.
Impact
Compile errors or silent miscompiles in user harnesses.
Suggested fix
Drop the trailing ;, prefix internal helpers with __tsffs_, rename the macro local, dedupe identical headers.
Related
#394, #379 (arm32 index)
Status
Reported by static code review at 79972db; not yet re-verified line by line. Found by reading the code; not reproduced under SIMICS.
- 主要言語
- Rust
- スター
- 335
- フォーク
- 26
- 平均マージ
- 1日 50分
- マージ済み PR（30日）
- 12
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
intel/tsffs のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
bug UI
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
wardian-app/Wardian#1746 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
jlcodes99/cockpit-tools#3125 ·
メンテナーはふだん 2 日以内に返信
-
bug user-sprint
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
Vondor/KassenSturz#967 ·
メンテナーはふだん 1 日以内に返信
-
area/ci
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
[BUG] Incremental sync warns "no stored highest_uid" on every round for folders with no new mailオープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
rustmailer/bichon#411 ·