diff options
| author | Andrei Golubev <andrei.golubev@qt.io> | 2021-11-22 13:55:38 +0100 |
|---|---|---|
| committer | Andrei Golubev <andrei.golubev@qt.io> | 2021-11-23 18:01:13 +0100 |
| commit | ac2e7839b05bc0fa343c3ae266a8e53610aa11e6 (patch) | |
| tree | 99fd08e9e02ae1638a1a86a887bacd1b318b8060 /src/qmlcompiler/qqmljsscope_p.h | |
| parent | 147b659a9427718c30bf3774aa1ef33804e97465 (diff) | |
Further improve QQmlJSMetaPropertyBindings (1/N)
Clean up QQmlJSScope property binding API:
- make hasPropertyBinding() plural (as its "own" counterpart)
- remove propertyBinding(name). it is convenient to use except
when it's wrong. luckily we don't seem to use it in qtdeclarative
Use this API instead of accessing m_propertyBindings directly in the
internal code
Make QQmlJSMetaPropertyBinding constructors explicit, because they really
should be
Change-Id: Ie72d1e0204422465ba36c2d0b15450b5445068f8
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
Diffstat (limited to 'src/qmlcompiler/qqmljsscope_p.h')
| -rw-r--r-- | src/qmlcompiler/qqmljsscope_p.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/qmlcompiler/qqmljsscope_p.h b/src/qmlcompiler/qqmljsscope_p.h index 16e3f1722c..2bb23ebf63 100644 --- a/src/qmlcompiler/qqmljsscope_p.h +++ b/src/qmlcompiler/qqmljsscope_p.h @@ -258,8 +258,7 @@ public: return m_propertyBindings.contains(name); } - bool hasPropertyBinding(const QString &name) const; - QQmlJSMetaPropertyBinding propertyBinding(const QString &name) const; + bool hasPropertyBindings(const QString &name) const; QList<QQmlJSMetaPropertyBinding> propertyBindings(const QString &name) const; static QQmlJSScope::ConstPtr ownerOfProperty(const QQmlJSScope::ConstPtr &self, |