From 3ac329cfbd50ca62ef40530a093db9462295c58a Mon Sep 17 00:00:00 2001 From: NaySurGithub <96527285+NaySurGithub@users.noreply.github.com> Date: Sat, 7 Nov 2026 00:14:55 +0100 Subject: [PATCH] fix(test): keep heldbuild::Hit's brace order for existing initialisers #72 inserted 'assist' right after 'block', so heldbuild_selftest's Hit{true, {{0, 0, 0}, 5}, {1, 0.5f, 0.5f}} fed the target into the bool and every CI build failed on that test. 'assist' now comes last. --- src/client/player/HeldBuild.hpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/client/player/HeldBuild.hpp b/src/client/player/HeldBuild.hpp index 9f6d0e7..87596e1 100644 --- a/src/client/player/HeldBuild.hpp +++ b/src/client/player/HeldBuild.hpp @@ -21,9 +21,10 @@ namespace perlin::heldbuild { struct Hit { bool block = false; // crosshair on a block (direct hit) - bool assist = false; // the block is the look-down pick under the player, not a ray hit Target target; glm::vec3 point{}; // where the ray stopped: hit point, or eye + look * reach without a hit + // last: callers brace-initialise the fields above in order + bool assist = false; // the block is the look-down pick under the player, not a ray hit }; class HeldBuild {