RESOLVED FIXED 230889
Fix rounding issue in run-jsc-stress-tests 
https://bugs.webkit.org/show_bug.cgi?id=230889
Summary Fix rounding issue in run-jsc-stress-tests
Geza Lore
Reported 2021-09-28 04:10:54 PDT
Fix rounding issue in run-jsc-stress-tests
Attachments
Patch (1.28 KB, patch)
2021-09-28 04:17 PDT, Geza Lore 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Geza Lore
Comment 1 2021-09-28 04:17:51 PDT
Created attachment 439455 [details] Patch
Geza Lore
Comment 2 2021-09-28 04:22:09 PDT
Sometimes ceil(N/K) == ceil(N/(K-1)), plus Ruby Array.slice gives you a nil if the start offset is past the end. This fixes the crashes arising when you did hit that rare combination of N (number of tests) and K (number of parallel processes).
EWS
Comment 3 2021-09-28 05:44:17 PDT
Committed r283164 (242212@main): <https://commits.webkit.org/242212@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 439455 [details].
Radar WebKit Bug Importer
Comment 4 2021-09-28 05:45:16 PDT
<rdar://problem/83619026>
Note You need to log in before you can comment on or make changes to this bug.