C is stupid

This commit is contained in:
Robbe Van Herck 2022-02-10 17:43:45 +01:00
parent 3a21da7d25
commit 91f6c4bab4

View file

@ -145,6 +145,7 @@ bool loopPuzzle(obus_can::message* message, void (*callback_game_start)(uint8_t
if (obus_can::receive(message)) { if (obus_can::receive(message)) {
received_message = true; received_message = true;
if (is_from_controller(message->from)) { if (is_from_controller(message->from)) {
uint32_t seed;
switch (message->msg_type) { switch (message->msg_type) {
case OBUS_MSGTYPE_C_GAMESTART: case OBUS_MSGTYPE_C_GAMESTART:
if (acked_after_last_hello) { if (acked_after_last_hello) {
@ -176,7 +177,7 @@ bool loopPuzzle(obus_can::message* message, void (*callback_game_start)(uint8_t
break; break;
case OBUS_MSGTYPE_C_INFOSTART: case OBUS_MSGTYPE_C_INFOSTART:
// Add module type and id to seed, to remove correlation in randomness between modules // Add module type and id to seed, to remove correlation in randomness between modules
uint32_t seed = message->infostart.seed + ((uint32_t) this_module.type << 8) + ((uint32_t) this_module.id); seed = message->infostart.seed + ((uint32_t) this_module.type << 8) + ((uint32_t) this_module.id);
// randomSeed has no effect when called with 0 as seed, so we use // randomSeed has no effect when called with 0 as seed, so we use
// a fallback value that is unlikely to collide with other frequently used seeds // a fallback value that is unlikely to collide with other frequently used seeds
if (seed == 0) { if (seed == 0) {