<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 22, 2017 at 11:30 PM, Armin Le Grand <span dir="ltr"><<a href="mailto:armin_le_grand@me.com" target="_blank">armin_le_grand@me.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hiho,<br>
<br>
here are some cleanup start points:<br>
<br>
- m_bUseCanvas: no longer needed, no longetr interesting from outside Metafile<br>
- ImplPlayWithRenderer: should be removed. Adding a warning in Metafile::Play for comments with GDIPLUS which guides to use primitive renderer for better handling would be nice to add<br>
- MtfRenderer: The hard part. com\sun\star\rendering\MtfRend<wbr>erer.idl and com\sun\star\rendering\MtfRend<wbr>erer.idl and their should be removed. It's not an official API and should be safe to remove<br>
- there may be more as consequence of these...as always ;-)<br>
<br>
Of course some UnitTests would be nice, too, best with EMF+ test data files which make use of the new stuff.<br>
<br>
As Thorsten already stated: Help much appreciated!<br>
<br>
Regards,<br>
<br>
Armin (alg)</blockquote><div><br></div><div><br></div><div>How hard would it be to move the workbench example in VCL to use this?<div><br></div><div><a href="https://cgit.freedesktop.org/libreoffice/core/tree/vcl/workben/mtfdemo.cxx">https://cgit.freedesktop.org/libreoffice/core/tree/vcl/workben/mtfdemo.cxx</a></div></div><div><br></div><div> </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="gmail-h5">
Am 22.08.2017 um 12:48 schrieb Thorsten Behrens:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Next steps (again, help much appreciated!):<br>
<br>
- add more GraphicPrimitive generators, complete up to a state that<br>
   the filter at least covers what the current direct canvas renderer<br>
   delivers:<br>
    * clip regions<br>
    * text rendering - it's parsed, but graphic primitive generation is<br>
      missing<br>
    * gradients/hatches<br>
<br>
</blockquote>
That's now done to a point where we're ~on par with the old renderer's<br>
functionality - thx to Patrick, Noel and Armin for all the nice work!<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
- once this works, cut over to new impl, bin the old code in cppcanvas<br>
   (there's sadly a small amount of duplicate code now in the parsers)<br>
- iteratively expand, to cover near-100% of EMF+ (mostly lacking around<br>
   gradients & effects):<br>
    * more complex fill types like bitmaps/tiles<br>
    * path gradient<br>
    * xor & stuff<br>
<br>
</blockquote>
That cut-over has now happened with<br>
<br>
<a href="https://cgit.freedesktop.org/libreoffice/core/commit/?id=ebc11ae0b132eefd3b1b1a837a8d0ad3ba73b460" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/l<wbr>ibreoffice/core/commit/?id=ebc<wbr>11ae0b132eefd3b1b1a837a8d0ad3b<wbr>a73b460</a><br>
<br>
, also killing the functionality to compare the two renderers - if you<br>
still need that, it's a rather smallish revert for now.<br>
<br>
Next up is cleansing the old renderer under cppcanvas, I suspect<br>
there's some easy wins there for easy hackers. :)<br>
<br>
Cheers,<br>
<br>
-- Thorsten<br>
</blockquote>
<br></div></div>
--<br>
ALG (PGP Key: EE1C 4B3F E751 D8BC C485 DEC1 3C59 F953 D81C F4A2)<br>
<br>
______________________________<wbr>_________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.<wbr>org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/libreoffice</a><br>
</blockquote></div><br></div></div>