Turn on error logging at all times
authorSøren Sandmann Pedersen <ssp@redhat.com>
Thu, 31 Jan 2013 19:36:38 +0000 (14:36 -0500)
committerSøren Sandmann Pedersen <ssp@redhat.com>
Wed, 13 Feb 2013 07:18:01 +0000 (02:18 -0500)
commit4dced81c917c753a4e699e3793efa15a39361cf0
treee3c91d87cafd4fc26385b114ece54bc349077dce
parentf4c9492c12d98f76d99b4dbdca56d517e1ffdb19
Turn on error logging at all times

While releasing 0.29.2 the distcheck run produced a number of error
messages that had to be fixed in 349015e1fc5d912ba4253133b90e751d0b.
These were not caught before so nobody had actually run pixman with
debugging turned on. It's not the first time this has happened, see
5b0563f39eb29e4ae431717696174da5 for example.

So this patch makes the return_if_fail() macros use unlikely() around
the expressions and then turns on error logging at all times. The
performance hit should negligible since we were already evaluating the
expressions.

The place where DEBUG actually does cause a performance hit is in the
region selfcheck code, and that will still only be enabled in
development snapshots.
pixman/pixman-private.h
pixman/pixman-utils.c