Remove QJSValue::propertyFlags() function
authorKent Hansen <kent.hansen@nokia.com>
Tue, 17 Jan 2012 14:23:01 +0000 (15:23 +0100)
committerQt by Nokia <qt-info@nokia.com>
Thu, 2 Feb 2012 07:10:24 +0000 (08:10 +0100)
commit54c51c89e21800b3a455127b3c19a3c5b30ee319
tree9781a998567abf4513919cc08e928a3df03eecbf
parented84a6ee63ac7a53f37efad3eb4a7e4eaa047242
Remove QJSValue::propertyFlags() function

Rationale: The API is incomplete. There is no way to set/update the
flags; adding a third (optional) argument to setProperty() is ugly.

If necessary, a QJSPropertyDescriptor API should be added, to enable
complete setting/querying of properties.

The flags were moved to QJSValuePrivate, and the propertyFlags()
implementation kept, because there is one place the read-only flag
is checked internally (in qv8typewrapper).

Task-number: QTBUG-23604
Change-Id: I089dce28dbda428662aac7c8d1536987fe6d855c
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@nokia.com>
13 files changed:
src/declarative/qml/v8/qjsconverter_impl_p.h
src/declarative/qml/v8/qjsconverter_p.h
src/declarative/qml/v8/qjsvalue.cpp
src/declarative/qml/v8/qjsvalue.h
src/declarative/qml/v8/qjsvalue_impl_p.h
src/declarative/qml/v8/qjsvalue_p.h
src/declarative/qml/v8/qscriptoriginalglobalobject_p.h
src/declarative/qml/v8/qv8engine_impl_p.h
src/declarative/qml/v8/qv8engine_p.h
src/declarative/qml/v8/qv8typewrapper.cpp
tests/auto/declarative/qjsengine/tst_qjsengine.cpp
tests/auto/declarative/qjsvalue/tst_qjsvalue.cpp
tests/auto/declarative/qjsvalue/tst_qjsvalue.h