Re: [PATCH 03/41] core: _canonical_box() allow NULL box ptr

"Enrico Weigelt, metux IT consult" <[email protected]>
Newsgroups gmane.comp.lib.cairo
Message-ID <[email protected]>
On 12.12.2015 10:50, Uli Schlachter wrote:

> Ah, thanks for the info. I'd suggest adding "A later patch will use this with a
> NULL argument", or something like that.

okay, added it.

> Can't you just use a non-NULL argument here? 

would mean having an unnecesary copy ...

> Or, instead add an explicit NULL
> check to _cairo_path_fixed_is_box() so that it does not call _canonical_box() at
> all?

That's a good idea. Will be coming in next iteration.


--mtx

--
Enrico Weigelt,
metux IT consulting
+49-151-27565287
-- 
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.