JeffersonLab / sim-recon

Simulation and Reconstruction for GlueX
9 stars 14 forks source link

Safe-guard against int wrap-around #1187

Closed aaust closed 6 years ago

aaust commented 6 years ago

If the RF bunch is close to the maximum of a 32 bit integer, the analysis library could get stuck in an infinite loop. This is a simple safe guard. In parallel, the BCal code was modified by a different commit to prevent crazy RF times from the start.

gluex commented 6 years ago

Test status for this pull request: SUCCESS

Summary: /work/halld/pull_request_test/sim-recon^aaust_analysis_loop_safe/tests/summary.txt Logs: /work/halld/pull_request_test/sim-recon^aaust_analysis_loop_safe/tests/log

Build log: /work/halld/pull_request_test/sim-recon^aaust_analysis_loop_safe/make_aaust_analysis_loop_safe.log Build report: /work/halld/pull_request_test/sim-recon^aaust_analysis_loop_safe/report_aaust_analysis_loop_safe.txt Location of build: /work/halld/pull_request_test/sim-recon^aaust_analysis_loop_safe