evntsqd.m

[Stuart Anderson]

On Sun, 18 Dec 2005, Jamey Sharp wrote:

> Oh! OK, thanks for telling me, and I apologize for committing
> prematurely. I assumed that since I'd gotten no response after I posted
> the bug report and patch to Bugzilla three months ago, that nobody but
> me cared... :-)

Jamey,
 	There are a few people around that have some experince w/ this
test suite,but they are only on this list, and not monitoring the
bugzilla, thus part of the reason for having a different system here.

> I was committing the patch from this bug, which I think should be quite
> unobjectionable:
> 	https://bugs.freedesktop.org/show_bug.cgi?id=4394

So backing up a step or two, you are proposing a change to the strategy
which is documented in the .m file. Discussion should occur at that level
first, and once the strategy is agreed upon, then the actual code change.
For simple stuff, proposing both parts at the same time is probably
fine.

At a minimum, the strategy "pseudo-code" should be updated as well.

Does any one have any objections to the proposed change? or any
suggestions how this could be handled w/o loosing the test? Does
anything break by Xlib not actually queuing anything? Is a spec
change needed to clarify that XEventsQueued() may never see anything 
in the queue?


Please start a seperate thread on the list for each of these other bugs,
and let's hope that not everyone has checked out for the holidays yet.


>
> I would also appreciate consideration of this bug:
> 	https://bugs.freedesktop.org/show_bug.cgi?id=4387
> I have a proposed patch there but the XCloseDisplay fix doesn't quite
> work, and I'm not sure how to correctly fix the problem.
>
> I found one other bug in the test suite, but I have no fix for it:
> 	https://bugs.freedesktop.org/show_bug.cgi?id=4385
>
> Thanks!
>
> --Jamey
>


                                 Stuart

Stuart R. Anderson                               anderson at netsweng.com
Network & Software Engineering                   http://www.netsweng.com/
1024D/37A79149:                                  0791 D3B8 9A4C 2CDC A31F
                                                  BD03 0A62 E534 37A7 9149