Archived issue #0031619
Samples - update JNI sample to use Android Studio
Description
It is desired updating JNI sample for building it using Android Studio (recommended by Google) instead of Eclipse.
Steps to reproduce
N/A
Public activity
12 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Author.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Sat Jun 20 02:14:51 2020 +0300
0031619: Samples - update JNI sample to use Android Studio
Project structure and project files have been updated to use Android Studio 4.0, gradle and CMake.
Redundant OcctJni_Window has been replaced by Aspect_NeutralWindow.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Sat Jun 20 02:14:51 2020 +0300
0031619: Samples - update JNI sample to use Android Studio
Project structure and project files have been updated to use Android Studio 4.0, gradle and CMake.
Redundant OcctJni_Window has been replaced by Aspect_NeutralWindow.
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
Patch is ready for review in OCCT branch CR31619.
1. OcctJniActivity.java:
are the commented rows necessary?
//int isAlreadyGranted = myContext.checkSelfPermission (thePermission);
//boolean toShowInfo = shouldShowRequestPermissionRationale(thePermission);
//requestPermissions (new String[]{thePermission}, 0);
2. /samples/java/jniviewer/app/src/main/jni/CMakeLists.txt
are the commented rows necessary?
#add_library(lib_FreeImage SHARED IMPORTED)
...
3. /samples/java/jniviewer/gradle.properties.template
are the path rows should give the custom path, why 7.5.0?
OCCT_ROOT=c\:/android/occt-7.5.0
...
is the commented row necessary?
#FREEIMAGE_ROOT=c\:/android/freeimage-3.17-android
4. /samples/java/jniviewer/ReadMe.md
You might need re-entering Android SDK explicitly in File -> Project Structure -> SDK Locations settings.
(and NDK Location setting)
are the commented rows necessary?
//int isAlreadyGranted = myContext.checkSelfPermission (thePermission);
//boolean toShowInfo = shouldShowRequestPermissionRationale(thePermission);
//requestPermissions (new String[]{thePermission}, 0);
2. /samples/java/jniviewer/app/src/main/jni/CMakeLists.txt
are the commented rows necessary?
#add_library(lib_FreeImage SHARED IMPORTED)
...
3. /samples/java/jniviewer/gradle.properties.template
are the path rows should give the custom path, why 7.5.0?
OCCT_ROOT=c\:/android/occt-7.5.0
...
is the commented row necessary?
#FREEIMAGE_ROOT=c\:/android/freeimage-3.17-android
4. /samples/java/jniviewer/ReadMe.md
You might need re-entering Android SDK explicitly in File -> Project Structure -> SDK Locations settings.
(and NDK Location setting)
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
> 1. OcctJniActivity.java: are the commented rows necessary?
Removed.
> 2. are the commented rows necessary?
Yes, in case if OCCT was built with optional FreeImage library enabled.
> 3. why 7.5.0?
This is just a template. Renamed to occt-dev-android.
> 4. /samples/java/jniviewer/ReadMe.md
Amended.
Removed.
> 2. are the commented rows necessary?
Yes, in case if OCCT was built with optional FreeImage library enabled.
> 3. why 7.5.0?
This is just a template. Renamed to occt-dev-android.
> 4. /samples/java/jniviewer/ReadMe.md
Amended.
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.
Number of compiler warnings:
No new/fixed warnings
Regressions/Differences/Improvements:
No regressions/differences
CPU differences:
Debian80-64:
OCCT
Total CPU difference: 17164.36000000016 / 17160.400000000114 [+0.02%]
Products
Total CPU difference: 11173.720000000081 / 11197.400000000112 [-0.21%]
Windows-64-VC14:
OCCT
Total CPU difference: 18707.65625 / 18650.578125 [+0.31%]
Products
Total CPU difference: 13062.6875 / 13012.71875 [+0.38%]
Image differences :
No differences that require special attention
Memory differences :
No differences that require special attention
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.
Number of compiler warnings:
No new/fixed warnings
Regressions/Differences/Improvements:
No regressions/differences
CPU differences:
Debian80-64:
OCCT
Total CPU difference: 17164.36000000016 / 17160.400000000114 [+0.02%]
Products
Total CPU difference: 11173.720000000081 / 11197.400000000112 [-0.21%]
Windows-64-VC14:
OCCT
Total CPU difference: 18707.65625 / 18650.578125 [+0.31%]
Products
Total CPU difference: 13062.6875 / 13012.71875 [+0.38%]
Image differences :
No differences that require special attention
Memory differences :
No differences that require special attention
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]