Phase 3: -Wreorder を潰して -Werror=reorder を有効化

[thawk105]

#43 Phase 3 のサブタスク。-Wreorder (20 件ユニーク) を潰し、ccbench_add_protocol() に -Werror=reorder を追加する。

修正概要

すべて TxExecutor (10 protocols) と TxNode (oze) のコンストラクタで member initializer list の順序がクラス内のメンバ宣言順と一致していなかったケース。C++ の初期化は宣言順で行われるので、initializer list の順は無視される。「見た目の順に初期化されている」と読み手が思い込むのを防ぐため、宣言順に並べ替え。

各箇所でメンバ初期化式に副作用が無いことを確認 (純粋なリファレンス bind / コピー / TxScanCallback(this) のような前提のない式) してから順序のみを変更。挙動の変化は無し。

修正ファイル

箇所	クラス
cc/cicada/include/transaction.hh:65	TxExecutor
cc/d2pl/include/transaction.hh:60	TxExecutor
cc/ermia/include/transaction.hh:57	TxExecutor
cc/mocc/include/transaction.hh:62-63	TxExecutor (callback_ が先頭宣言)
cc/mvto/include/transaction.hh:58	TxExecutor
cc/oze/include/oze.hh:40-49	TxNode (2 コンストラクタ)
cc/oze/include/transaction.hh:75	TxExecutor
cc/si/include/transaction.hh:57	TxExecutor
cc/silo/include/transaction.hh:69	TxExecutor
cc/ss2pl/include/transaction.hh:46	TxExecutor
cc/tictoc/include/transaction.hh:58	TxExecutor

真のバグ

無し。順序変更で挙動が変わるパターン (Foo(int x) : a_(x++), b_(x) のような副作用持ち) は混じっていなかった。

CMake

 target_compile_options(${target} PRIVATE
   -Werror=maybe-uninitialized
   -Werror=unused-but-set-variable
-  -Werror=unused-label)
+  -Werror=unused-label
+  -Werror=reorder)

Test plan

• GCC 13 Release / -DENABLE_SANITIZER=OFF: 34/34
• GCC 11 Debug + ASan: 34/34
• CI 緑

Related

• 段階的に -Werror 化してビルドを堅牢にする #43 Phase 3 (parallel work)
• Phase 1: -Wunused-label を潰して -Werror=unused-label を有効化 (Phase 1 完了) #50 (Phase 1 完了)