[cairo] [PATCH 05/51] core: helper function for converting cairo_status_t to human-readable text (like strerror)
Bryce Harrington
bryce at osg.samsung.com
Tue Dec 22 14:19:46 PST 2015
On Fri, Dec 18, 2015 at 09:36:29PM +0100, Enrico Weigelt, metux IT consult wrote:
> On 18.12.2015 20:44, Bill Spitzak wrote:
> > I was figuring that functions are not supposed to ever return
> > LAST_STATUS as an error, so it is just as wrong as returning
> > LAST_STATUS+1 and should display in a similar way.
>
> hmm, I dont really have an oppinion on that. Would it make any
> practical difference ?
>
> The function is just meant for debug outputs, so one does not need
> to look up the enum values manually.
The function is defined as returning a const char*, so printing a
textual string rather than a number is what it should be doing.
Getting the numeric value of an enum parameter instead of a readable
name is rarely the problem you're dealing with during debugging.
Usually it's the opposite. :-)
> --mtx
>
> --
> Enrico Weigelt,
> metux IT consulting
> +49-151-27565287
> --
> cairo mailing list
> cairo at cairographics.org
> http://lists.cairographics.org/mailman/listinfo/cairo
More information about the cairo
mailing list