proposition for patch in CommonSalLayout.cxx

julien2412 serval2412 at
Tue Mar 9 17:34:21 UTC 2021

Stanisław Jeśmanowicz wrote
> ...
> Thank you for your advice, but getting familiar with gerrit on top of git
> would take some time for 
> me now.
> I hope, that some of your developers could commit this patch for me in vcl
> module.
> It is simple and won't harm anything.
> But people could benefit from it on all platforms that use system's native
> HarfBuzz implementation.
> ...

I can understand you don't have time for gerrit but would it be possible you
submit a license statement?

About this line, I retrieved the initial patches which put this line:
1) earliest
Author: Khaled Hosny <khaledhosny at>
Date:   Fri Mar 10 16:53:08 2017 +0200

    tdf#106466: Use graphite2 shaper first
    We want to always prefer Graphite shaping when supported by the font,
    which is also what HarfBuzz does by default.
    Change-Id: I6670fc03b8e6b3d7e07e1b8e0062880524da1655
    Tested-by: Jenkins <ci at>
    Reviewed-by: Khaled Hosny <khaledhosny at>

2) oldest
Author: Khaled Hosny <khaledhosny at>
Date:   Thu Oct 6 04:15:41 2016 +0200

    Use HarfBuzz shape plan for a bit more control
    This way we control exactly what shapers we use in what order, and as an
    extra we can now tell which shaper HarfBuzz ends up using.


Sent from:

More information about the LibreOffice mailing list