Skip to content

add waitKeyEx method#692

Merged
justadudewhohacks merged 2 commits into
justadudewhohacks:masterfrom
Garfonso:master
May 13, 2020
Merged

add waitKeyEx method#692
justadudewhohacks merged 2 commits into
justadudewhohacks:masterfrom
Garfonso:master

Conversation

@Garfonso

Copy link
Copy Markdown
Contributor

Needed on windows (and maybe other systems?) to capture special buttons (like arrow keys).

See for example here: https://stackoverflow.com/questions/45580975/opencv-handling-arrow-keys-with-waitkey-function

Needed on windows (and maybe other systems?) to capture special buttons (like arrow keys).
Comment thread cc/io/io.cc
Comment thread cc/io/io.cc
Comment thread cc/io/io.cc
Comment thread cc/io/io.cc
Comment thread cc/io/io.h
@@ -16,6 +16,7 @@ class Io {
static NAN_METHOD(Imshow);
static NAN_METHOD(ImshowWait);
static NAN_METHOD(WaitKey);

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

#if CV_VERSION_GREATER_EQUAL(3, 2, 0)

Comment thread cc/io/io.h
@justadudewhohacks

Copy link
Copy Markdown
Owner

Thanks and sorry for the delay! It looks like cv::waitKeyEx has been introduced with opencv3.2.0. Could you add the version guard for lower versions (see my comments).

@Garfonso

Copy link
Copy Markdown
Contributor Author

Thanks and sorry for the delay! It looks like cv::waitKeyEx has been introduced with opencv3.2.0. Could you add the version guard for lower versions (see my comments).

Hi, sorry, I somehow have not been notified of your comments. Thanks for the feedback.
I will try to add those and update the PR.

@justadudewhohacks

Copy link
Copy Markdown
Owner

Sorry for the delay, and thanks for the change!

@justadudewhohacks justadudewhohacks merged commit 86c6fc0 into justadudewhohacks:master May 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants