On Sat, Sep 8, 2012 at 3:59 AM, Peter Samuelson <[email protected]> wrote:
> > > for (; len > 0; ++data, --len) > > - if (*data < 0) > > - return data; > > + if (*data < 0 || *data >= 0x80) > > + return data; > > A reasonable compiler will collapse it anyway, but this is shorter and > more direct: > > if (*data & 0x80) > You are right. I'm just using the same pattern as for the "<=" variant. -- Stefan^2. -- * Join us this October at Subversion Live 2012<http://www.wandisco.com/svn-live-2012> for two days of best practice SVN training, networking, live demos, committer meet and greet, and more! Space is limited, so get signed up today<http://www.wandisco.com/svn-live-2012> ! *

