Archived issue #0031619

Samples - update JNI sample to use Android Studio

Open CASCADEOCCT:Samplesclosed12 public notes

Search issues

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.

01Commenter 1
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.
02Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
03Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
04Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
05Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
06Author
Patch is ready for review in OCCT branch CR31619.
07Commenter 2
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)
08Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
09Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
10Author
> 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.
11Commenter 3
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
12Commenter 1
Branch [archived branch] has been deleted by Participant.

[revision removed]