[cairo] Conditionally include debug functions with ENABLE_DEBUG_FUNCS

Behdad Esfahbod behdad at behdad.org
Thu Oct 18 14:33:43 PDT 2007


On Thu, 2007-10-18 at 17:13 -0400, BJörn Lindqvist wrote:
> Hello,
> 
> Here is a patch that introduces a ENABLE_DEBUG_FUNCS macro define. Its
> purpose is to allow for debug stuff to be conditionally included in
> Cairo proper. Things that must be exposed for testing to work, but
> that should not be included in the final and really public API.
> 
> An example is the cairo_boilerplate_xlib_surface_disable_render()
> function. It is one of those functions that it is much easier to deal
> with if it is inside Cairo rather than outside since it must mock
> around with Cairo internals to work.
> 
> Other debug functions than cairo_xlib_surface_disable_render() can be
> added in the future.

So, what's the justification for this?  Boilerplate functions used to be
in libcairo and it took me two or three days of work to move them out to
get to the clean state that it is currently.  Moving them back is just
regressing.

Lets not add more "state" to the cairo build.

-- 
behdad
http://behdad.org/

"Those who would give up Essential Liberty to purchase a little
 Temporary Safety, deserve neither Liberty nor Safety."
        -- Benjamin Franklin, 1759





More information about the cairo mailing list