_pixman_run_fast_path: typo
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 28 Aug 2009 10:31:06 +0000 (06:31 -0400)
committerSøren Sandmann Pedersen <sandmann@redhat.com>
Fri, 28 Aug 2009 10:43:39 +0000 (06:43 -0400)
This is one example of a compiler warning that was lost amit the build
noise.

The error here is that in a list of required conditions we used ';'
instead of '&&' with the result of continuing to use the fast-path
even if we had a wide mask.

Another error is that it was testing src, not mask as it should.

pixman/pixman-utils.c

index 961c6c6..7128206 100644 (file)
@@ -670,15 +670,16 @@ _pixman_run_fast_path (const pixman_fast_path_t *paths,
 
     if (mask && has_fast_path)
     {
-       has_fast_path = mask->type == BITS &&
-                       !mask->common.transform &&
-                       !mask->common.alpha_map &&
-                       !mask->bits.read_func &&
-                       !mask->bits.write_func &&
-                        mask->common.filter != PIXMAN_FILTER_CONVOLUTION &&
-                        mask->common.repeat != PIXMAN_REPEAT_PAD &&
-                        mask->common.repeat != PIXMAN_REPEAT_REFLECT;
-                       !PIXMAN_FORMAT_IS_WIDE (src->bits.format);
+       has_fast_path =
+           mask->type == BITS &&
+           !mask->common.transform &&
+           !mask->common.alpha_map &&
+           !mask->bits.read_func &&
+           !mask->bits.write_func &&
+           mask->common.filter != PIXMAN_FILTER_CONVOLUTION &&
+           mask->common.repeat != PIXMAN_REPEAT_PAD &&
+           mask->common.repeat != PIXMAN_REPEAT_REFLECT &&
+           !PIXMAN_FORMAT_IS_WIDE (mask->bits.format);
     }
 
     if (has_fast_path)