Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Java VideoCapture buffered stream constructor#27284

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
asmorkalov merged 17 commits intoopencv:4.xfromdkurt:java_video_capture_read
May 14, 2025

Conversation

@dkurt
Copy link
Member

Pull Request Readiness Checklist

resolves#26809

See details athttps://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

asmorkalov reacted with thumbs up emoji
@dkurtdkurt changed the titleJava VideoCapture buffered streamJava VideoCapture buffered stream constructorMay 5, 2025
@asmorkalovasmorkalov self-requested a reviewMay 5, 2025 16:37
@asmorkalovasmorkalov self-assigned thisMay 5, 2025
@dkurtdkurtforce-pushed thejava_video_capture_read branch from9b46002 tofaddefcCompareMay 6, 2025 15:13
@dkurtdkurtforce-pushed thejava_video_capture_read branch 5 times, most recently from80b985b to6dfcd4eCompareMay 7, 2025 13:40
@dkurtdkurtforce-pushed thejava_video_capture_read branch from6dfcd4e to79afad9CompareMay 7, 2025 13:59
@dkurtdkurt marked this pull request as ready for reviewMay 7, 2025 14:29
@dkurtdkurt requested a review fromopencv-alalekMay 8, 2025 06:20
@asmorkalov
Copy link
Contributor

Great job!

@asmorkalov
Copy link
Contributor

Android build error:

 ccache /opt/android-sdk/ndk/26.3.11579264/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=armv7-none-linux-androideabi21 --sysroot=/opt/android-sdk/ndk/26.3.11579264/toolchains/llvm/prebuilt/linux-x86_64/sysroot -DCVAPI_EXPORTS -D_USE_MATH_DEFINES -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/home/ci/opencv/modules/core/misc/java/src/cpp -I/home/ci/opencv/modules/imgproc/misc/java/src/cpp -I/home/ci/opencv/modules/ml/misc/java/src/cpp -I/home/ci/opencv/modules/photo/misc/java/src/cpp -I/home/ci/opencv/modules/dnn/misc/java/src/cpp -I/home/ci/opencv/modules/features2d/misc/java/src/cpp -I/home/ci/opencv/modules/imgcodecs/misc/java/src/cpp -I/home/ci/opencv/modules/videoio/misc/java/src/cpp -I/home/ci/opencv/modules/calib3d/misc/java/src/cpp -I/home/ci/opencv/modules/objdetect/misc/java/src/cpp -I/home/ci/opencv/modules/video/misc/java/src/cpp -I/home/ci/opencv/modules/java/generator/src/cpp -I/home/ci/build/o4a/modules/java_bindings_generator/gen/cpp -I/home/ci/opencv/modules/core/include -I/home/ci/opencv/modules/flann/include -I/home/ci/opencv/modules/imgproc/include -I/home/ci/opencv/modules/ml/include -I/home/ci/opencv/modules/photo/include -I/home/ci/opencv/modules/dnn/include -I/home/ci/opencv/modules/features2d/include -I/home/ci/opencv/modules/imgcodecs/include -I/home/ci/opencv/modules/videoio/include -I/home/ci/opencv/modules/calib3d/include -I/home/ci/opencv/modules/objdetect/include -I/home/ci/opencv/modules/video/include -isystem /home/ci/build/o4a -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -march=armv7-a -mthumb -Wformat -Werror=format-security     -fsigned-char -W -Wall -Wreturn-type -Wnon-virtual-dtor -Waddress -Wsequence-point -Wformat -Wformat-security -Wmissing-declarations -Wmissing-prototypes -Wstrict-prototypes -Wundef -Winit-self -Wpointer-arith -Wshadow -Wsign-promo -Wuninitialized -Winconsistent-missing-override -Wno-delete-non-virtual-dtor -Wno-unnamed-type-template-args -Wno-comment -Wno-deprecated-enum-enum-conversion -Wno-deprecated-anon-enum-enum-conversion -fdiagnostics-show-option -Qunused-arguments  -fvisibility=hidden -fvisibility-inlines-hidden  -Wno-deprecated-declarations -O3 -DNDEBUG   -DNDEBUG -fPIC -std=c++11 -MD -MT modules/java/jni/CMakeFiles/opencv_java.dir/__/__/videoio/misc/java/src/cpp/videoio_converters.cpp.o -MF modules/java/jni/CMakeFiles/opencv_java.dir/__/__/videoio/misc/java/src/cpp/videoio_converters.cpp.o.d -o modules/java/jni/CMakeFiles/opencv_java.dir/__/__/videoio/misc/java/src/cpp/videoio_converters.cpp.o -c /home/ci/opencv/modules/videoio/misc/java/src/cpp/videoio_converters.cpp/home/ci/opencv/modules/videoio/misc/java/src/cpp/videoio_converters.cpp:11:39: error: cannot initialize a parameter of type 'JNIEnv **' (aka '_JNIEnv **') with an rvalue of type 'void **'        res = vm->AttachCurrentThread((void**)&env, NULL);                                      ^~~~~~~~~~~~/opt/android-sdk/ndk/26.3.11579264/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/include/jni.h:1057:39: note: passing argument to parameter 'p_env' here    jint AttachCurrentThread(JNIEnv** p_env, void* thr_args)                                      ^1 error generated.
dkurt reacted with thumbs up emoji

@dkurtdkurtforce-pushed thejava_video_capture_read branch fromd129930 tof9cb6d4CompareMay 13, 2025 12:05
@dkurtdkurtforce-pushed thejava_video_capture_read branch from7ee4ad3 to797d610CompareMay 14, 2025 07:51
@dkurtdkurtforce-pushed thejava_video_capture_read branch from797d610 toc3be053CompareMay 14, 2025 07:51
@asmorkalov
Copy link
Contributor

@opencv-alalek Do you have other remarks? Is it ready for merge?

Copy link
Contributor

@opencv-alalekopencv-alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Well done 👍

dkurt, asmorkalov, and utibenkei reacted with heart emoji
@asmorkalovasmorkalov merged commit67ba045 intoopencv:4.xMay 14, 2025
54 of 55 checks passed
@dkurtdkurt deleted the java_video_capture_read branchMay 14, 2025 15:31
@asmorkalovasmorkalov mentioned this pull requestMay 27, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@asmorkalovasmorkalovasmorkalov approved these changes

@opencv-alalekopencv-alalekopencv-alalek approved these changes

Assignees

@asmorkalovasmorkalov

Projects

None yet

Milestone

4.12.0

Development

Successfully merging this pull request may close these issues.

No way to initialize the IStreamReader class in the Java wrapper.

3 participants

@dkurt@asmorkalov@opencv-alalek
[8]ページ先頭
©2009-2025 Movatter.jp