Re: [Libevent-users] Event priorities

2012-04-30 Thread Ralph Castain
On Apr 30, 2012, at 7:49 PM, Nick Mathewson wrote: > On Mon, Apr 30, 2012 at 7:17 PM, Ralph Castain wrote: > [...] >> The branch looks like it does the trick, and I like that it also does not >> force the re-scan unless the newly activated event is a higher priority. >> >> +1 from me. >> >> M

Re: [Libevent-users] Event priorities

2012-04-30 Thread Nick Mathewson
On Mon, Apr 30, 2012 at 7:17 PM, Ralph Castain wrote: [...] > The branch looks like it does the trick, and I like that it also does not > force the re-scan unless the newly activated event is a higher priority. > > +1 from me. > > Meantime, I gather this was against the current 2.0 trunk as oppo

Re: [Libevent-users] Event priorities

2012-04-30 Thread Ralph Castain
On Apr 30, 2012, at 5:10 PM, Nick Mathewson wrote: > On Mon, Apr 30, 2012 at 6:45 PM, Ralph Castain wrote: > [...] >> I'll take a look at your branch, but had some further thoughts in the >> interim. Would a more optimal option be to add the following logic to >> event_priority_set: >> >> eve

Re: [Libevent-users] Event priorities

2012-04-30 Thread Nick Mathewson
On Mon, Apr 30, 2012 at 6:45 PM, Ralph Castain wrote: [...] > I'll take a look at your branch, but had some further thoughts in the > interim. Would a more optimal option be to add the following logic to > event_priority_set: > > event_priority_set(ev, pri) { >    IF AND >        IF base> EQ

Re: [Libevent-users] Event priorities

2012-04-30 Thread Ralph Castain
On Apr 30, 2012, at 4:38 PM, Nick Mathewson wrote: > On Mon, Apr 30, 2012 at 4:48 PM, Ralph Castain wrote: >> >> On Apr 30, 2012, at 2:30 PM, Nick Mathewson wrote: >> >>> On Mon, Apr 30, 2012 at 3:53 PM, Ralph Castain wrote: Hi folks I'm working with priorities in events using

Re: [Libevent-users] Event priorities

2012-04-30 Thread Nick Mathewson
On Mon, Apr 30, 2012 at 4:48 PM, Ralph Castain wrote: > > On Apr 30, 2012, at 2:30 PM, Nick Mathewson wrote: > >> On Mon, Apr 30, 2012 at 3:53 PM, Ralph Castain wrote: >>> Hi folks >>> >>> I'm working with priorities in events using libevent 2.0.13. Since I'm not >>> using the most current relea

Re: [Libevent-users] Event priorities

2012-04-30 Thread Ralph Castain
On Apr 30, 2012, at 2:30 PM, Nick Mathewson wrote: > On Mon, Apr 30, 2012 at 3:53 PM, Ralph Castain wrote: >> Hi folks >> >> I'm working with priorities in events using libevent 2.0.13. Since I'm not >> using the most current release, I thought I'd ask about a behavior we are >> seeing that m

Re: [Libevent-users] Event priorities

2012-04-30 Thread Nick Mathewson
On Mon, Apr 30, 2012 at 3:53 PM, Ralph Castain wrote: > Hi folks > > I'm working with priorities in events using libevent 2.0.13. Since I'm not > using the most current release, I thought I'd ask about a behavior we are > seeing that might be a potential bug. It's not the cause of your problem

Re: [Libevent-users] Event priorities

2012-04-30 Thread Vincent Bernat
OoO En ce début de soirée du lundi 30 avril 2012, vers 21:53, Ralph Castain disait : > I'm working with priorities in events using libevent 2.0.13. Since I'm > not using the most current release, I thought I'd ask about a behavior > we are seeing that might be a potential bug. > I have atta

[Libevent-users] Event priorities

2012-04-30 Thread Ralph Castain
Hi folks I'm working with priorities in events using libevent 2.0.13. Since I'm not using the most current release, I thought I'd ask about a behavior we are seeing that might be a potential bug. I have attached a simple reproducer of the problem. In short, suppose I have setup 8 priorities fo