alex/baseview
1
0
Fork 0
Code Activity

Merge pull request #132 from robbert-vdh/fix/x11-frame-pacing

Browse source 
Ensure a consistent frame pacing in X11 windows
This commit is contained in:
william light 2022-11-18 18:06:11 +01:00 committed by GitHub
parent 1780c89baf 13cc12d4e7
commit 84f10763a3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
src/x11/window.rs
View file

@ -445,19 +445,21 @@ impl Window {
xcb::ffi::xcb_get_file_descriptor(raw_conn)
};
let mut next_frame = Instant::now() + self.frame_interval;
let mut last_frame = Instant::now();
self.event_loop_running = true;
while self.event_loop_running {
let now = Instant::now();
let until_next_frame = if now > next_frame {
// We'll try to keep a consistent frame pace. If the last frame couldn't be processed in
// the expected frame time, this will throttle down to prevent multiple frames from
// being queued up. The conditional here is needed because event handling and frame
// drawing is interleaved. The `poll()` function below will wait until the next frame
// can be drawn, or until the window receives an event. We thus need to manually check
// if it's already time to draw a new frame.
let next_frame = last_frame + self.frame_interval;
if Instant::now() >= next_frame {
handler.on_frame(&mut crate::Window::new(self));
next_frame = Instant::now() + self.frame_interval;
self.frame_interval
} else {
next_frame - now
};
last_frame = Instant::max(next_frame, Instant::now() - self.frame_interval);
}
let mut fds = [PollFd::new(xcb_fd, PollFlags::POLLIN)];
@ -466,7 +468,8 @@ impl Window {
self.drain_xcb_events(handler);
// FIXME: handle errors
poll(&mut fds, until_next_frame.subsec_millis() as i32).unwrap();
poll(&mut fds, next_frame.duration_since(Instant::now()).subsec_millis() as i32)
.unwrap();
if let Some(revents) = fds[0].revents() {
if revents.contains(PollFlags::POLLERR) {