Comment 12 for bug 1391857

Revision history for this message
In , Henriqueleng-t (henriqueleng-t) wrote :

Hi Massimo. Looks like you last patch with only 3 lines of code solved the problem!
I applied here and no more segfault.
How i don't understand and i know nothing about cairo, i don't know if what this patch do is the right thing or if is safe use it. All i know it is that it solved, i think.

Should i mark as solved?

I apllyied here and
(In reply to comment #4)
> Created attachment 104568 [details] [review]
> alternative solution
>
> An alternative patch that would make the rectangular tessellator
> work also with empty boxes, so intrinsically safe also for
> other (future) callers.
>
> Obviously only valid if I understood what it's doing and it
> is possible to effectively ignore empty boxes.
>
> > Would it be ok with you if this test case were included with Cairo? Would the > following license header be suitable? Do you want some copyright notice to be > included? Which one?
>
> it is ok with me, it is suitable, don't want any copyright notice, whatever.