Siarhei Siamashka <[email protected]> writes: > @@ -2590,6 +2589,8 @@ PREFIX (_init_from_image) (region_type_t *region, > > PREFIX(_init) (region); > > + assert (region->data); > + > return_if_fail (image->type == BITS); > return_if_fail (image->bits.format == PIXMAN_a1);
I'm a little uncomfortable with adding asserts because if they trigger, they will take down the X server. (Even though this one can't possibly trigger). Instead, I'd rather use the critical_if_fail() macro. If pixman is compiled with DEBUG defined, it will print stuff to stderr if the given expression fails. Soren _______________________________________________ Pixman mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pixman
