Skip to content
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.

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

Skip processing of output render callbacks with 0 frames #228

Merged
merged 2 commits into from
Jun 26, 2024

Conversation

Pehrsons
Copy link
Contributor

@Pehrsons Pehrsons requested a review from padenot June 25, 2024 08:44
@Pehrsons Pehrsons force-pushed the no-empty-data-callback branch from 52299fe to 908814c Compare June 25, 2024 19:39
The current HOST_TIME_TO_NS_RATIO implemented with lazy_static is
causing what appears to be false-positive TSAN errors. OnceLock and
LazyLock exhibit similar errors under TSAN.

It seemed easier to rewrite host_time_to_ns_ratio to be a init-once
context member, rather than making the unstable `no_sanitize` attribute
work. Being unstable it also needed to be included conditionally only
when compiled with the nightly compiler. And trying the rustversion
crate for this resulted in compilation failure due to TSAN reporting it
doesn't work...
@Pehrsons Pehrsons force-pushed the no-empty-data-callback branch from 908814c to 8400cfa Compare June 25, 2024 20:50
@Pehrsons Pehrsons merged commit 8bce3b3 into trailblazer Jun 26, 2024
8 checks passed
@Pehrsons Pehrsons deleted the no-empty-data-callback branch June 26, 2024 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants