[patch] event_loopbreak()
Marc Lehmann
schmorp at schmorp.de
Tue Jul 7 03:28:02 CEST 2009
On Tue, Jul 07, 2009 at 12:22:16AM +0400, Antony Dovgal <tony at daylessday.org> wrote:
> I'd like to propose a patch adding support for event_loopbreak(), event_base_loopbreak() and ev_break():
> http://dev.daylessday.org/diff/libev_event_loopbreak.diff
hmm, you also add some (unexplained) functionality to ev.c - I assume
event_break is some new libevent functionality?
Since I couldn't quickly find any useful documentation on these new
functions, and you didn't provide one, what are they supposed to do?
libevent documents them as:
Abort the active event_base_loop() immediately.
event_base_loop() will abort the loop after the next event is completed
Your patch, howver, implements ev_unloop in a different way, so it seems
superfluous.
Assuming your patch is flawed and your intention was to implement the
function as dcoumented by libevent, I am still reluctant to slow down one
of the central inner loops, and libev seems to be strong enough nowadays
as to not require full libevent api support anymore, so, what's a use case
for this, I cannot imagine any?
> I didn't actually use or see libev until today, so bear with me if the patch is wrong/incomplete.
That's fine :) I still need more explanations before applying patches in
general, though.
(I assume the patch is a proof-of-concept patch, if I were to accept
the feature, it would need documenttaion, proper formatting, and the
MULTIPLICITY changes need to be removed - but I still need a good reason
for this functionality).
Greetings,
--
The choice of a Deliantra, the free code+content MORPG
-----==- _GNU_ http://www.deliantra.net
----==-- _ generation
---==---(_)__ __ ____ __ Marc Lehmann
--==---/ / _ \/ // /\ \/ / pcg at goof.com
-=====/_/_//_/\_,_/ /_/\_\
More information about the libev
mailing list