diff options
author | A. Wilcox <AWilcox@Wilcox-Tech.com> | 2023-01-31 01:58:32 -0600 |
---|---|---|
committer | A. Wilcox <AWilcox@Wilcox-Tech.com> | 2023-01-31 01:58:32 -0600 |
commit | acc601bde0d1f241e5683d8fd1e365bf60d9d52f (patch) | |
tree | e3ce74fbc5bbcd6094b8f907f53142b0211056b3 /system/sp | |
parent | 90dc8905f7dbd3d0290232b378539d29e96ee49b (diff) | |
download | packages-acc601bde0d1f241e5683d8fd1e365bf60d9d52f.tar.gz packages-acc601bde0d1f241e5683d8fd1e365bf60d9d52f.tar.bz2 packages-acc601bde0d1f241e5683d8fd1e365bf60d9d52f.tar.xz packages-acc601bde0d1f241e5683d8fd1e365bf60d9d52f.zip |
user/grantlee: Explicitly register enum comparator
This is necessary to make the tests pass on 64-bit PowerPC. This is
caused because the MetaEnumVariable class does not register its
comparators, which is required in Qt 5.
Qt 6 will use introspection to automatically do this. The proper fix is
likely to use a constructor function somewhere in the Grantlee library
to ensure it is called on library initialisation, but it isn't
immediately obvious to me where to put such a ctor.
This only seems to affect 64-bit PowerPC, as other platforms do pass
this test; maybe Qt is doing some sort of introspection that doesn't
work on ppc64. It is also affected by GCC optimisation flags, so it may
be a miscompilation.
At any rate, this patch *does* fix the tests. Prior to 5.3.0, enums did
not compare correctly anyway, so this does not have a very high impact.
Fixes: #955
Diffstat (limited to 'system/sp')
0 files changed, 0 insertions, 0 deletions