[gst-devel] [gst-cvs] gstreamer: micro-optim: if (x) is cheaper than if (x > 0) for unsigned integers

Tim-Philipp Müller t.i.m at zen.co.uk
Fri Dec 3 12:47:19 CET 2010


On Fri, 2010-12-03 at 03:04 -0800, Edward Hervey wrote:

> Module: gstreamer
> Branch: master
> Commit: 6aa8ca37eeb9debfa6919741a023250bf278248f
> URL:    http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=6aa8ca37eeb9debfa6919741a023250bf278248f
> 
> Author: Edward Hervey <bilboed at bilboed.com>
> Date:   Sat Apr 11 15:04:41 2009 +0200
> 
> micro-optim: if (x) is cheaper than if (x > 0) for unsigned integers

I really really dislike this change. Is it really worth it?

I think this compromises code readability too much.

Cheers
 -Tim






More information about the gstreamer-devel mailing list