[PATCH weston v3 08/11] Change boolean repaint_scheduled to quad-state enum

Daniel Stone daniel at fooishbar.org
Fri Mar 10 14:28:14 UTC 2017


Hi Pekka,

On 8 March 2017 at 13:22, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> On Fri,  3 Mar 2017 16:59:42 +0000 Daniel Stone <daniels at collabora.com> wrote:
>> repaint_scheduled is actually cleverly a quad-state, disguised as a
>> boolean. There are four possible conditions for the repaint loop to be
>> in at any time:
>>   - loop idle; no repaint will occur until specifically requested, which
>>     may be never (repaint_scheduled == 0)
>>   - loop schedule to begin: the loop was previously idle, but due to a
>>     repaint-schedule request, we will call the start_repaint_loop hook
>>     in the next idle timer
>
> s/timer/task/
>
> I can fix that when merging. Right?

Yes please!

Cheers,
Daniel


More information about the wayland-devel mailing list