> On 3 Jul 2018, at 05:55, Thiago Macieira <[email protected]> wrote:
> 
> On Saturday, 30 June 2018 10:09:11 PDT Lars Knoll wrote:
>> Hi Thiago,
>> 
>> maybe https://codereview.qt-project.org/#/c/233693/ helps with this case.
> 
> It did help some.  It went from
> 
> PASS   : tst_QComplexText::bidiTest(line #201771 (LTR)
> 
> to
> 
> PASS   : tst_QComplexText::bidiTest(line #456371 (RTL))
> 
> So we doubled the performance. But it wasn't enough.

Are you sure you already have the fix (it went into 5.11 first)? Because with 
the patch, you shouldn't see those 'line #xxx' things at all anymore. bidiTest 
should just report one single line.

Cheers,
Lars

> 
> Trying Eddy's suggestion instead:
>       https://codereview.qt-project.org/233794
> 
> -- 
> Thiago Macieira - thiago.macieira (AT) intel.com
>  Software Architect - Intel Open Source Technology Center
> 
> 
> 
> _______________________________________________
> Development mailing list
> [email protected]
> http://lists.qt-project.org/mailman/listinfo/development

_______________________________________________
Development mailing list
[email protected]
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to