Yes, will revert that patch and backport the correct one.

Thanks
vivek

On Mon, Mar 6, 2023 at 10:47 AM DC <[email protected]> wrote:

> The patches submitted to
> dunfell ->
> https://lists.openembedded.org/g/openembedded-core/message/177768
> Kirkstone ->
> https://lists.openembedded.org/g/openembedded-core/message/177767
> <https://lists.openembedded.org/g/openembedded-core/message/177767>
>
> Still exhibits O(n^2) behaviour upon testing with applications and fonts
> and in worst cases Harfbuzz is not printing anything.
> Upon closer examination, looks to me that a wrong fix is pushed for the
> issue.
>
> The patch [
> https://github.com/harfbuzz/harfbuzz/commit/85be877925ddbf34f74a1229f3ca1716bb6170dc
> ] is known to causes erroneous results and regression already reverted by
> Behdad as a developer of Harfbuzz in the commit [
> https://github.com/harfbuzz/harfbuzz/commit/661050b4659ee490dfe622821bc7fde7d1c40510
> ].
>
> (ref->
> https://github.com/harfbuzz/harfbuzz/commit/85be877925ddbf34f74a1229f3ca1716bb6170dc#commitcomment-101326711
> )
>
> The new fix was already introduced (
> https://github.com/harfbuzz/harfbuzz/commit/85be877925ddbf34f74a1229f3ca1716bb6170dc#commitcomment-101335712)
> and needs to be backported properly.
>
> It doesn't look correct to patch Harfbuzz with code already reverted.
>
> 
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#178057): 
https://lists.openembedded.org/g/openembedded-core/message/178057
Mute This Topic: https://lists.openembedded.org/mt/97419449/21656
Mute 
#kirkstone:https://lists.openembedded.org/g/openembedded-core/mutehashtag/kirkstone
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to