Re: [Libevent-users] [patch] more testcases

2011-10-21 Thread Nick Mathewson
On Thu, Oct 20, 2011 at 4:51 AM, Leonid Evdokimov wrote: > On Thu, Oct 20, 2011 at 06:52, Nick Mathewson wrote: >> On Wed, Oct 19, 2011 at 9:58 AM, Leonid Evdokimov wrote: > So it looks like bug in pinning/unpinning events to the self-debugging > hashmap. > > Be the way - maybe, all tests shou

Re: [Libevent-users] [patch] more testcases

2011-10-20 Thread Leonid Evdokimov
On Thu, Oct 20, 2011 at 06:52, Nick Mathewson wrote: > On Wed, Oct 19, 2011 at 9:58 AM, Leonid Evdokimov wrote: > (The style tweak is that I generally do >  "struct foo *bar;" rather than "struct foo* bar;" Yep, I know that "*" applies to specific variable and not to the type, that's just my hab

Re: [Libevent-users] [patch] more testcases

2011-10-19 Thread Nick Mathewson
On Wed, Oct 19, 2011 at 9:58 AM, Leonid Evdokimov wrote: > Here are some patches. > > 0001 fires without following fixes I like it; applying with a few comments and style tweaks. (The style tweak is that I generally do "struct foo *bar;" rather than "struct foo* bar;" because in C syntax, the

[Libevent-users] [patch] more testcases

2011-10-19 Thread Leonid Evdokimov
Here are some patches. 0001 fires without following fixes aff6ba1 Fix request_finished memory leak with debugging turned on. 9b724b2 Fix evsig_dealloc memory leak with debugging turned on. 0002 is just a preparation for 0003 0003 fires with current codebase but, seems, it should not. I'm sorry,