[CinCV TNG] [PATCH] Rewrite of BC_DisplayInfo::test_window

Johannes Sixt j6t at kdbg.org
Mon Apr 27 20:47:56 CEST 2015


Am 26.04.2015 um 22:23 schrieb Einar R√ľnkaru:
> Hi.
>
> On 04/25/2015 11:16 PM, Johannes Sixt wrote:
>>
>> Increasing the wait time to 0.1sec helps indeed. But I dislike that
>> there is a dependency on the timing. Are we guaranteed to receive a
>> ConfigureNotify event?
>
> I think that one or more is guaranteed. Your fix does not work for me.
> On my case 3 ConfigureNotify events are sent and the last contains right
> coordinates.
>
> I do not like the experimntal sleep time too. But I didn't get to work
> it on my system without it.
>
> Can you test the attached new version of the patch?

The patch works.

But is the usleep still required? The break condition of the loop is the 
DestroyNotify, and XNextEvent() will wait for an event anyway. For me, 
it works when I remove the usleep.

BTW, the new 'break;' is not indented correctly.

-- Hannes



More information about the Cinelerra mailing list