[cairo-bugs] [Bug 73580] cairo-perf-trace failure

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=73580

--- Comment #12 from Andrea Canciani <ranma42 at gmail.com> ---
Comment on attachment 110953
  --> https://bugs.freedesktop.org/attachment.cgi?id=110953
[PATCH] cairo-trace: Extract common helper for
_surface_create_similar[_image]()

Review of attachment 110953:
-----------------------------------------------------------------

I expressed some concerns about this refactoring on IRC, but I'll also post
them here to keep track of them together with the bug.

While removing some code duplication between similar and similar-image is
ideally good, in practice the two operators have different signatures (they use
different parameters in a different order), so I'm afraid that coalescing both
in a single function makes them more complicated than needed.
I agree that the changes from c1ac8db7e60bcec8235b09729b34154e95790534 should
also apply to similar-image, but I believe that merging the emission of the two
operator is not a good idea.

Ideally I would merge the previous patch (which fixes the
cairo_surface_create_similar_image() function so that it does not pushes more
surfaces than expected on the stack) and (in a separate commit) resync the
implementation of similar and similar-image, possibly by only sharing the code
for _emit_other_obj().

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cairographics.org/archives/cairo-bugs/attachments/20150103/7c449de0/attachment.html>