nanogui: Thread: Possible bug in GrCheckNextEvent() in nanox/client_ecos.c

In the function GrCheckNextEvent(), the client-side event queue is not
checked at all, while the function GrGetNextEventTimeout() DO check this
client-side event queue.
So, an event previously fetched in client-side queue will not be seen by the
GrCheckNextEvent!
To fix this inconsistancy, the following lines should be inserted in the
first line in GrCheckNextEvent() (in nanox/client.c)

/*Checking client side queue before querying server */
if (evlist) { GetNextQueuedEvent(ep); CheckErrorEvent(ep); return; }

Regards,
I-Jui Sung

Good eye.  Is that broken in the regular nanox/client.c too???

On Wednesday 22 August 2001 16:30, I-Jui Sung mentioned:
> In the function GrCheckNextEvent(), the client-side event queue is not
> checked at all, while the function GrGetNextEventTimeout() DO check this
> client-side event queue.
> So, an event previously fetched in client-side queue will not be seen by
> the GrCheckNextEvent!
> To fix this inconsistancy, the following lines should be inserted in the
> first line in GrCheckNextEvent() (in nanox/client.c)
>
> /*Checking client side queue before querying server */
> if (evlist) { GetNextQueuedEvent(ep); CheckErrorEvent(ep); return; }
>
> Regards,
> I-Jui Sung
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: ####@####.####
> For additional commands, e-mail: ####@####.####

Gee, I meant nanox/client.c..(Actually I found this bug in my custom eCos
port of microwindows..), but I've checked it in the CVS version of
microwindows, it's broken too.

Happy hacking,
I-Jui Sung
----- Original Message -----
From: "Jordan Crouse" ####@####.####
To: "I-Jui Sung" ####@####.#### ####@####.####
Sent: Thursday, August 23, 2001 6:21 AM
Subject: Re: [nanogui] Possible bug in GrCheckNextEvent() in
nanox/client_ecos.c


> Good eye.  Is that broken in the regular nanox/client.c too???
>
> On Wednesday 22 August 2001 16:30, I-Jui Sung mentioned:
> > In the function GrCheckNextEvent(), the client-side event queue is not
> > checked at all, while the function GrGetNextEventTimeout() DO check this
> > client-side event queue.
> > So, an event previously fetched in client-side queue will not be seen by
> > the GrCheckNextEvent!
> > To fix this inconsistancy, the following lines should be inserted in the
> > first line in GrCheckNextEvent() (in nanox/client.c)
> >
> > /*Checking client side queue before querying server */
> > if (evlist) { GetNextQueuedEvent(ep); CheckErrorEvent(ep); return; }
> >
> > Regards,
> > I-Jui Sung
> >
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: ####@####.####
> > For additional commands, e-mail: ####@####.####
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: ####@####.####
> For additional commands, e-mail: ####@####.####
>