vectorの要素数が更新されないバグ

[windfall-shogi]

YaneuraOu/source/engine/dlshogi-engine/YaneuraOu_dlshogi_bridge.cpp

Lines 376 to 387 in 599378d

	result.clear();
	result.reserve(num);
	for (ChildNumType i = 0; i < num; ++i)
	{
	auto& child = root_node->child[i];
	Move m = child.move;
	// move_count == 0であって欲しくはないのだが…。
	float win = child.move_count == 0 ? child.nnrate : (float)child.win / child.move_count;
	// result.emplace_back(std::pair<Move, float>(m, win));
	result[i] = std::pair<Move, float>(m, win);
	}

386行目では要素数が更新されないので、バグと思われる。
385行目をコメントアウトしているので、何か意図があるようにも思うがわからない。

[yaneurao]

それ、377行目でreserve()してます。事前に必要な要素数分を確保することによって、emplace_back()でvectorの自動拡張(これはallocしなおしになる)が行われるのを防止してるんです。

[yaneurao]

ああ、reserveではなくresizeと書くべきということですね。うっかりしてました。

ただ、reserveと書いてもend()が違うだけで配列の要素はメモリ上にリニアに確保されてますんで、問題なく書き込めますし(bounds checkerみたいなので警告はでるかも)、コンストラクタとデストラクタが呼び出されないだけっすね。今回の場合、この構造体にデストラクタは存在しないので問題ナッシングという…。

しかし気持ち悪いので修正しました。ご指摘ありがとうございました。
9f6284f