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

various fixes #1242

Merged
merged 3 commits into from
Apr 17, 2024
Merged

various fixes #1242

merged 3 commits into from
Apr 17, 2024

Conversation

absolutelynothelix
Copy link
Collaborator

backend/gl: specify the time uniform location in blit_shader_glsl

to avoid breaking custom animated shaders by having to specify it
themselves.

picom: fix the benchmark option

mimic the normal rendering routine by setting ps->render_queued to true
before setting and starting the draw timer.

picom: fix custom animated shaders

the draw timer has to be stopped before calling the draw_callback_impl
function because when there's a custom animated shader it calls the
queue_redraw funtion that calls the schedule_render function that sets
and starts the draw timer that gets immediately stopped otherwise.

this fixes the screen not being redirected back with unredir-if-possible
and a custom animated shader and only the first frame being drawn with
vsync, no-frame-pacing and a custom animated shader.

Copy link
Owner

@yshui yshui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so glad you found these... 👍

to avoid breaking custom animated shaders by having to specify it
themselves.
mimic the normal rendering routine by setting ps->render_queued to true
before setting and starting the draw timer.
the draw timer has to be stopped before calling the draw_callback_impl
function because when there's a custom animated shader it calls the
queue_redraw funtion that calls the schedule_render function that sets
and starts the draw timer that gets immediately stopped otherwise.

this fixes the screen not being redirected back with unredir-if-possible
and a custom animated shader and only the first frame being drawn with
vsync, no-frame-pacing and a custom animated shader.
@yshui yshui merged commit 1c72c26 into yshui:next Apr 17, 2024
14 checks passed
@absolutelynothelix absolutelynothelix deleted the various-fixes branch April 17, 2024 12:52
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