Good catch. Our primary build platforms are Linux and Mac, which is why we didn't trip into this.
- Doug On Mon, Mar 29, 2010 at 11:34 PM, Mateusz Berezecki <[email protected]>wrote: > Hi, > > I'm doing minor code reviews and here's the patch for IOHandler.cc > > diff --git a/src/cc/AsyncComm/IOHandler.cc b/src/cc/AsyncComm/IOHandler.cc > index 19d95e1..8e6dde1 100644 > --- a/src/cc/AsyncComm/IOHandler.cc > +++ b/src/cc/AsyncComm/IOHandler.cc > @@ -195,7 +195,7 @@ int IOHandler::add_poll_interest(int mode) { > int IOHandler::remove_poll_interest(int mode) { > > if ((m_poll_interest & mode) == 0) > - return; > + return Error::OK; > > m_poll_interest &= ~mode; > > -- > You received this message because you are subscribed to the Google Groups > "Hypertable Development" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]<hypertable-dev%[email protected]> > . > For more options, visit this group at > http://groups.google.com/group/hypertable-dev?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Hypertable Development" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/hypertable-dev?hl=en.
