Add support for naming JNI functions with the unstringify macro

The manually defined native JNI function tables allow defining a name
that may be different from the actual function name; this name is then
used from the Java-side.

This can be useful to provide also as an option with the new
"unstringifying" macros which can help for example in porting code
to these new macros.

Pick-to: 6.4
Change-Id: Icfebfb351cb8dfb122795d20b37e2eac167a41bf
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>

2 files changed, 27 insertions, 0 deletions

diff --git a/tests/auto/corelib/kernel/qjnienvironment/testdata/src/org/qtproject/qt/android/testdata/QtJniEnvironmentTestClass.java b/tests/auto/corelib/kernel/qjnienvironment/testdata/src/org/qtproject/qt/android/testdata/QtJniEnvironmentTestClass.java
index 0fc70f2567..342342c548 100644
--- a/tests/auto/corelib/kernel/qjnienvironment/testdata/src/org/qtproject/qt/android/testdata/QtJniEnvironmentTestClass.java
+++ b/tests/auto/corelib/kernel/qjnienvironment/testdata/src/org/qtproject/qt/android/testdata/QtJniEnvironmentTestClass.java
@@ -6,6 +6,7 @@ package org.qtproject.qt.android.testdatapackage;
 public class QtJniEnvironmentTestClass
 {
     private static native void callbackFromJava(String message);
+    private static native void namedCallbackFromJava(String message);
     private static native void intCallbackFromJava(int value);
 
     public final int INT_FIELD = 123;
@@ -18,6 +19,11 @@ public class QtJniEnvironmentTestClass
         callbackFromJava("From Java: " + message);
     }
 
+    public static void namedAppendJavaToString(String message)
+    {
+        namedCallbackFromJava("From Java (named): " + message);
+    }
+
     public static void convertToInt(String message)
     {
         intCallbackFromJava(Integer.parseInt(message));
diff --git a/tests/auto/corelib/kernel/qjnienvironment/tst_qjnienvironment.cpp b/tests/auto/corelib/kernel/qjnienvironment/tst_qjnienvironment.cpp
index 76cad4ddc0..4809505f47 100644
--- a/tests/auto/corelib/kernel/qjnienvironment/tst_qjnienvironment.cpp
+++ b/tests/auto/corelib/kernel/qjnienvironment/tst_qjnienvironment.cpp
@@ -101,6 +101,13 @@ static void callbackFromJava(JNIEnv *env, jobject /*thiz*/, jstring value)
 }
 Q_DECLARE_JNI_NATIVE_METHOD(callbackFromJava);
 
+static void tediouslyLongNamed_callbackFromJava(JNIEnv *env, jobject /*thiz*/, jstring value)
+{
+    Q_UNUSED(env)
+    registerNativesString = QJniObject(value).toString();
+}
+Q_DECLARE_JNI_NATIVE_METHOD(tediouslyLongNamed_callbackFromJava, namedCallbackFromJava)
+
 static void callbackFromJavaNoCtor(JNIEnv *env, jobject /*thiz*/, jstring value)
 {
     Q_UNUSED(env)
@@ -126,6 +133,20 @@ void tst_QJniEnvironment::registerNativeMethods()
         QVERIFY(registerNativesString == QStringLiteral("From Java: Qt"));
     }
 
+    // Named native function
+    {
+        QVERIFY(env.registerNativeMethods(javaTestClass, {
+            Q_JNI_NATIVE_METHOD(tediouslyLongNamed_callbackFromJava)
+        }));
+
+        QJniObject::callStaticMethod<void>(javaTestClass,
+                                           "namedAppendJavaToString",
+                                           "(Ljava/lang/String;)V",
+                                            QtString.object<jstring>());
+        QTest::qWait(200);
+        QVERIFY(registerNativesString == QStringLiteral("From Java (named): Qt"));
+    }
+
     // No default constructor in class
     {
         QVERIFY(env.registerNativeMethods(javaTestClassNoCtor, {