6502/CLK
1
0
Fork 0
mirror of https://github.com/TomHarte/CLK.git synced 2025-03-15 00:34:09 +00:00
Code Issues Projects Releases Wiki Activity

Corrects potential JustInTimeActor overflow.

Browse Source
This commit is contained in:
Thomas Harte 2021-06-27 21:36:41 -04:00
parent 2f53b105bb
commit 5729e6e13a
1 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
ClockReceiver/JustInTime.hpp
View File

@ -228,7 +228,19 @@ template <class T, class LocalTimeScale = HalfCycles, int multiplier = 1, int di
/// Updates this template's record of the next sequence point.
void update_sequence_point() {
if constexpr (has_sequence_points<T>::value) {
time_until_event_ = object_.get_next_sequence_point() * divider;
// Keep a fast path where no conversions will be applied; if conversions are
// going to be applied then do a direct max -> max translation rather than
// allowing the arithmetic to overflow.
if constexpr (divider == 1 && std::is_same_v<LocalTimeScale, TargetTimeScale>) {
time_until_event_ = object_.get_next_sequence_point();
} else {
const auto time = object_.get_next_sequence_point();
if(time == TargetTimeScale::max()) {
time_until_event_ = LocalTimeScale::max();
} else {
time_until_event_ = time * divider;
}
}
assert(time_until_event_ > LocalTimeScale(0));
}
}