On Sat, Dec 22, 2007 at 06:18:12PM -0500, Jeff Squyres <[EMAIL PROTECTED]> 
wrote:
> >>static struct ev_loop *
> >>ev_default_loop (unsigned int flags) { ... }
> >>
> >>Should that function be marked as inline?
> >
> >No, because inline semantics are not portable at the moment (C99 vs.  
> >gcc),
> >so static is the closest equivalent.
> 
> Also FWIW: AC_C_INLINE can take of this for you -- here's its  
> description:

Jeff, please read what I wrote. You do not understand what inline means
and how C compilers generally work. I do not see how AC_C_INLINE is
relevant, as it doesn't tell me about inline _semantics_ and how to use
it, and it requires autoconf. Also, the current code and semantics are
*correct* and the wanted ones, and not using the inline function might
generate a warning as well, so silencing it in gcc is of little value.

If you are concerned, do not use -Wall in production builds (using -Wall
is basically a mistake in production builds as it includes an unknown,
often changing and sometimes broken set of warnings that will only scare
users). Please do not think that a warning indicates any problem with the
code whatsoever.

Because of our communications history, wouldn't it be possible to
"send" an actual developer who knows C and portability a bit? It is a
bit tiresome to have to explain C keywords and unix functions to you,
especially as portability issues require some experience, and my patience
admittedly has worn out a bit.

(Questions are ok, but if you do not like my answer because it doesn't
make sense to you you should first clarify it).

> >libev has been checked for relevant compiler warnings (on the  
> >platforms
> >and with the compilers I use only of course).
> 
> Fair enough.  Is OS X one of those platforms?  (I couldn't find an  
> official list of supported platforms -- did I miss it?)

OS X is not one of those, but since afaics only gcc is used the warnings
should be comparable to similar platforms.

The list of official platforms is:

- POSIX (all)
- Winsockets (Windows 2000+, maybe earlier)
- Perls select emulation on windows

Currently this apparently rules out OS X (which is utterly, dearly,
almost unusably broken), but libev does survive its testsuites there
(with select, the only working backend on OS X), so while not officially
supported (due to lakc of resources) it should generally work there.

> A related (but probably random) question: are there any correctness  
> tests for libev?

I use both the libevent regression tests and also have a number of tests
on my own (the latter are all part of the perl module).

> own stuff every night in an automated test harness; I'd be happy to  
> add libev (standalone; outside of the other stuff I test) to the mix  
> with a variety of platforms and compilers if it would be helpful.

Just run make test in the perl module, and if more is desired, use the
regression test in the libevent+libev tarball.

-- 
                The choice of a       Deliantra, the free code+content MORPG
      -----==-     _GNU_              http://www.deliantra.net
      ----==-- _       generation
      ---==---(_)__  __ ____  __      Marc Lehmann
      --==---/ / _ \/ // /\ \/ /      [EMAIL PROTECTED]
      -=====/_/_//_/\_,_/ /_/\_\

_______________________________________________
libev mailing list
libev@lists.schmorp.de
http://lists.schmorp.de/cgi-bin/mailman/listinfo/libev

Reply via email to