[cairo] Font options patch, first draft
cworth at redhat.com
Tue Jul 19 14:48:28 PDT 2005
On Mon, 18 Jul 2005 18:59:24 -0400, Owen Taylor wrote:
> On Tue, 2005-07-19 at 00:50 +0200, Torsten Schoenfeld wrote:
> > > void
> > > cairo_surface_get_font_options (cairo_surface_t *surface,
> > > cairo_font_options_t *options);
> > > ===
> > I'm not sure this would map well to languages with namespace support:
> > $surface->get_font_options ($options);
> > Getters usually don't modify their arguments -- they just return
> > something. I'm not really able to come up with a better name right now,
> > but I think it should be something with 'merge', 'update', 'overwrite'
> > or something along those lines.
> This would map to a straightforward getter in language bindings; just
> like the matrix functions and the font extents functions.
> $options = $surface->get_font_options ();
> It's uglier than the font extents functions and (reformed) matrix
> options because cairo_font_options_t can't be stack allocated but
> otherwise it follows the same pattern.
Did you consider modeling it after cairo_copy_path ?
> I didn't make it return a newly allocated object partially for this
> consistency and partially for performance reasons: to cut down
> just a tiny bit on the number of places where little blocks of
> memory are allocated and freed.
Huh? It's an opaque type so the application still has to allocate it
with cairo_font_options_create, right? (Though, I suppose the
application could pool font options, but I don't really see this
becoming a serious peformance issue).
To me it looks like the interface requires two functions
(cairo_font_options_create and cairo_surface_get_font_options) where
one would do just fine (cairo_surface_get_font_options).
Unless sharing the object with other calls is a natural part of the
common expected usage of this function. I'm not totally clear on that
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/cairo/attachments/20050719/48e24cdc/attachment.pgp
More information about the cairo