This is the mail archive of the cygwin@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] gettimeofday time travels V2


On Sat, Jul 06, 2002 at 11:55:17AM +0100, Philip Aston wrote:
>The list in thread.h is specific to a type, and intrusive. I used a
>mutex to create my own thread-safe, non-intrusive list. I used pthread
>mutexes instead of mutos since mutos must be allocated statically.

What is wrong with allocating a muto statically?  You only need one for
this application in cygwin.

>Christopher Faylor writes:
> > It may make sense to just make all of the members of the hires
> > class static since they are just maintaining global state.
>
>Agree. Personally I'd use a singleton object rather than static
>members which would also allow hires to continue to be a wm_listener,
>but it amounts to the same thing. However, I didn't do this as I
>consider it outside of the scope of the fix.

This looks like nice work but I am still concerned about the necessity
of starting up the windows thread and using the windows event loop as I
mentioned in previous email.  I'd rather not have that overhead if it
can be avoided.

For the cygwin part, the patch is also big enough that it requires an
assignment.  Hopefully you've looked at http://cygwin.com/contrib.html
by now...

The w32api part should be posted separately to cygwin-patches so that
Danny will see it and apply it.

cgf

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]