Handle invalid random target indices #144

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

themuffinator merged 1 commit into main from codex/replace-clamping-with-bounds-check-in-g_selectrandomtarget

Nov 22, 2025

Owner

themuffinator commented Nov 22, 2025

Summary

add bounds checking to G_SelectRandomTarget to log and reject invalid random indices
update target selection test to cover invalid RNG results while ensuring highest index can be chosen

Testing

g++ -std=c++17 src/tests/g_utils_target_selection_test.cpp -o /tmp/g_utils_target_selection_test && /tmp/g_utils_target_selection_test


          Handle invalid random target indices

6f7c91a

themuffinator added the codex label

— with

ChatGPT Codex Connector

themuffinator merged commit 107ceb8 into main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels