On Wed, 24 Mar 2010 10:42:33 -0600, Nathan Hjelm said:
>Looks good. A couple of things:
Thanks for looking!
>1) why are you storing the current runloop in event_thread_main? Run
>loops are associated with threads so the (current) two calls to
>CFRunLoopGetCurrent return the same value. There is no need to keep
>track of the current run loop.
The change solves a warning about unbalanced retain/release. The clang
static analyzer is not smart enough to realise that CFRunLoopGetCurrent
() returns the same thing both times.
>2) Apple uses a void * to pass the transfer size so it is probably safer
>to use 64 bits for io_size (certainly doesn't hurt).
That change was from Mike Colagrosso actually, see:
<http://old.nabble.com/-PATCH--Darwin%3A-Force-io_size-to-be-4-bytes-in-
darwin_async_io_callback-p27660019.html>
I think perhaps using uintptr_t would be better still.
>I have attached a
>patch making this change. Regenerate your patches versus the current
>tree patched with my change.
I'll try. :) I'm new to git...
--
____________________________________________________________
Sean McBride, B. Eng sean@...
Rogue Research http://www.rogue-research.com
Mac Software Developer Montréal, Québec, Canada
|