1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

JNI: make it safe to iterate over or convert an invalid QJniArray

Browse Source 
This is expected behavior for default-constructed containers.

Also add the missing isEmpty() member function to QJniArrayBase.

Pick-to: 6.8
Change-Id: I23111f6906ef5476567272cb23746fec962afa35
Reviewed-by: Assam Boudjelthia <assam.boudjelthia@qt.io>
This commit is contained in:
Volker Hilsheimer 2024-06-06 14:16:42 +02:00
parent 0255d3a9af
commit e5516191b2
2 changed files with 33 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/corelib/kernel/qjniarray.h
View File

@ -106,6 +106,7 @@ public:
template <typename T = jobject> template <typename T = jobject>
T object() const { return m_object.object<T>(); } T object() const { return m_object.object<T>(); }
bool isValid() const { return m_object.isValid(); } bool isValid() const { return m_object.isValid(); }
bool isEmpty() const { return size() == 0; }
size_type size() const size_type size() const
{ {
@ -310,6 +311,7 @@ public:
return res; return res;
} }
} }
auto toContainer() const auto toContainer() const
{ {
JNIEnv *env = jniEnv(); JNIEnv *env = jniEnv();
@ -328,11 +330,15 @@ public:
} else if constexpr (std::is_same_v<T, jbyte>) { } else if constexpr (std::is_same_v<T, jbyte>) {
const qsizetype bytecount = size(); const qsizetype bytecount = size();
QByteArray res(bytecount, Qt::Initialization::Uninitialized); QByteArray res(bytecount, Qt::Initialization::Uninitialized);
env->GetByteArrayRegion(object<jbyteArray>(), if (!isEmpty()) {
0, bytecount, reinterpret_cast<jbyte *>(res.data())); env->GetByteArrayRegion(object<jbyteArray>(),
0, bytecount, reinterpret_cast<jbyte *>(res.data()));
}
return res; return res;
} else { } else {
QList<T> res; QList<T> res;
if (isEmpty())
return res;
res.resize(size()); res.resize(size());
if constexpr (std::is_same_v<T, jchar>) { if constexpr (std::is_same_v<T, jchar>) {
env->GetCharArrayRegion(object<jcharArray>(), env->GetCharArrayRegion(object<jcharArray>(),

25
tests/auto/corelib/kernel/qjniarray/tst_qjniarray.cpp
View File

@ -17,6 +17,7 @@ public:
private slots: private slots:
void construct(); void construct();
void invalidArraysAreEmpty();
void size(); void size();
void operators(); void operators();
}; };
@ -126,6 +127,30 @@ void tst_QJniArray::construct()
} }
} }
void tst_QJniArray::invalidArraysAreEmpty()
{
QJniArray<jchar> invalid;
QVERIFY(!invalid.isValid());
QCOMPARE(invalid.object(), nullptr);
QVERIFY(invalid.isEmpty());
QCOMPARE(invalid.begin(), invalid.end());
QCOMPARE(invalid.rbegin(), invalid.rend());
QList<jchar> data;
// safe to iterate
for (const auto &e : invalid)
data.emplace_back(e);
QVERIFY(data.empty());
// safe to convert
data = invalid.toContainer();
QVERIFY(data.empty());
// unsafe to access
// auto element = invalid.at(0);
}
void tst_QJniArray::size() void tst_QJniArray::size()
{ {
QJniArray<jint> array; QJniArray<jint> array;