On Sun, Mar 5, 2023 at 7:36 PM vkumbhar <vkumb...@mvista.com> wrote: > > Yes, will revert that patch and backport the correct one.
I will remove that patch from the current kirkstone queue and await a V2. It hasn't moved out of my testing branch, so a revert isn't needed. Steve > On Mon, Mar 6, 2023 at 10:47 AM DC <davidco...@outlook.com> 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 >> >> 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 (#178072): https://lists.openembedded.org/g/openembedded-core/message/178072 Mute This Topic: https://lists.openembedded.org/mt/97419449/21656 Mute #kirkstone:https://lists.openembedded.org/g/openembedded-core/mutehashtag/kirkstone Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
