From d81626ece5e78c5727ffc72e9ed44f464c553ae2 Mon Sep 17 00:00:00 2001 From: mattzzw Date: Thu, 5 Mar 2026 08:22:22 +0100 Subject: [PATCH] Update examples/simple_repeater/MyMesh.cpp Fix pow() and rand() are calculated for every forwarded packet, but we only need it for flood advert. Also fixes 'paket' typo. Co-authored-by: Wessel --- examples/simple_repeater/MyMesh.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/examples/simple_repeater/MyMesh.cpp b/examples/simple_repeater/MyMesh.cpp index bd8b687f..069fad49 100644 --- a/examples/simple_repeater/MyMesh.cpp +++ b/examples/simple_repeater/MyMesh.cpp @@ -449,12 +449,13 @@ bool MyMesh::allowPacketForward(const mesh::Packet *packet) { } // Limit flood advert paket forwarding using a probabilistic reduction defined by P(h) = 0.308^(hops-1) // https://github.com/meshcore-dev/MeshCore/issues/1223 - double_t roll_dice = (double)rand() / RAND_MAX; - double_t forw_prob = pow(_prefs.flood_advert_base, packet->path_len - 1); - if (packet->getPayloadType() == PAYLOAD_TYPE_ADVERT && packet->isRouteFlood() && roll_dice > forw_prob) - return false; + if (packet->getPayloadType() == PAYLOAD_TYPE_ADVERT && packet->isRouteFlood()) { + double roll_dice = (double)rand() / RAND_MAX; + double forw_prob = pow(_prefs.flood_advert_base, packet->path_len - 1); + if (roll_dice > forw_prob) + return false; + } - // all other packets return true; }