[cairo] [cairo-commit] src/cairo.h src/cairo-scaled-font.c src/cairo-user-font.c
Behdad Esfahbod
behdad at behdad.org
Mon Sep 29 14:55:41 PDT 2008
Carl Worth wrote:
> On Mon, 2008-09-29 at 08:34 -0700, Behdad Esfahbod wrote:
>> New commits:
>> commit 1310df44edcf0b9fee648c8f8688d4a1177cde68
>> Author: Behdad Esfahbod <behdad at behdad.org>
>> Date: Mon Sep 29 11:34:19 2008 -0400
>>
>> [text_to_glyphs] Further enhance the docs
>>
>> The most complex public cairo API deserves more doc love...
> ...
>> if (*num_glyphs < 0) {
>> + if (orig_glyphs != *glyphs) {
>> + cairo_glyph_free (*glyphs);
>> + *glyphs = orig_glyphs;
>> + }
>> *num_glyphs = orig_num_glyphs;
>> return CAIRO_INT_STATUS_UNSUPPORTED;
>> }
>
> This code-change hunk obviously wasn't included in the commit-message,
Um, sorry, I just saw it after I read the commit mail.
> (and clearly wasn't intentional as part of the commit).
Could go as a separate commit, yeah. I was cross-referencing the docs and the
implementation and noted that I can be more pedantic there (since the other
side of this code can be a user-font text_to_glyphs that doesn't quite clean
after itself...)
> Still correct though? (Looks like it to me.)
Yes.
behdad
> -Carl
>
>
>
More information about the cairo
mailing list