[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1382204 - /subversion/trunk/subversion/libsvn_subr/utf_validate.c

From: Peter Samuelson <peter_at_p12n.org>
Date: Fri, 7 Sep 2012 20:59:44 -0500

> 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)

Peter
Received on 2012-09-08 04:00:25 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.