[Poppler-bugs] [Bug 105692] PDF compliance: Always respect Annot Rect

[bugzilla-daemon at freedesktop.org]

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

--- Comment #12 from Albert Astals Cid <aacid at kde.org> ---
(In reply to Tobias Deiminger from comment #11)
> Created attachment 138645 [details] [review]
> Apply template method design pattern to Annot::draw (Rev 4)
> 
> Continuous rebasing ¯\_(ツ)_/¯ I like that AnnotAppearanceBuilder...

I'm a bit conflicted with this patch.

I see it adds some structure, that is always nice, on the other hand it doesn't
really save that many lines as shown by the base implementations (that is what
we end up reusing) are basically one line each.

Also the new structure makes you add the fontResourceDict which is a bit harder
to manage than before since now it spans two methods instead of one.

On the other hand now we don't create the fonts dict on every draw that is a
good thing too (not that it's a hot path but all speedups are good)

Let me sleep over it :D

Comments either from you or from anyone else that add pros for this patch are
also welcome :)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/poppler-bugs/attachments/20180405/f45240d9/attachment-0001.html>