[Mesa-dev] [RFC 10/21] mesa: Remove equality check in helper functions

Erik Faye-Lund kusmabite at gmail.com
Wed Oct 21 23:43:43 PDT 2015


On Tue, Oct 20, 2015 at 12:44 AM, Nanley Chery <nanleychery at gmail.com> wrote:
> From: Nanley Chery <nanley.g.chery at intel.com>
>
> Since the version numbers being compared are integral and we don't ever
> expect gl_context::Version to be equal to 0, subtract 1 from the rhs of
> the equation and perform the optimization of removing the equality check.

Is this really an improvement? Changing '>=' to '>' and then making
all versions off-by-one seems like trading less readable and debugable
code for no real upside...


More information about the mesa-dev mailing list