diff options
author | Sami Shalayel <sami.shalayel@qt.io> | 2022-08-09 10:50:37 +0200 |
---|---|---|
committer | Sami Shalayel <sami.shalayel@qt.io> | 2022-08-10 00:48:31 +0200 |
commit | cec3b3b4f90a1db642b8c3e03c493ab7d6c5f8a5 (patch) | |
tree | c09635309c10920000078d6978e78d37c8f6bc56 /src/3rdparty/harfbuzz-ng/src/hb-ot-metrics.h | |
parent | 395c9ac731e3789f336bad09cb4a8918a73be43d (diff) |
Remove assertion from ownMethodIndex that breaks tests
ownMethodIndex works for all kind of methods, also for constructors.
Therefore, remove the assertion there (that checks for non constructors)
and add a test in qtbase so it does not happen again.
The test broken by the assertion is in qtdeclarative:
tst_QJSEngine::newQMetaObject().
Also rename QMetaMethodPrivate::ownConstructorIndex() to
ownConstructorMethodIndex() as the previous naming implied that
ownMethodIndex() could not be used for constructors.
amends b73ab954dffffc462b6f6efe5a2dd97efeab0038
Task-number: QTBUG-105360
Change-Id: I0244993ed79bee055645b5443f5d02e1c089a6c6
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'src/3rdparty/harfbuzz-ng/src/hb-ot-metrics.h')
0 files changed, 0 insertions, 0 deletions