Skip to content

fixed cv::moveWindow() on mac#16588

Merged
alalek merged 3 commits intoopencv:3.4from
vpisarev:fix_macos_move_window
Feb 17, 2020
Merged

fixed cv::moveWindow() on mac#16588
alalek merged 3 commits intoopencv:3.4from
vpisarev:fix_macos_move_window

Conversation

@vpisarev
Copy link
Copy Markdown
Contributor

@vpisarev vpisarev commented Feb 14, 2020

resolves #16343.

Thanks to @cwreynolds and @saskatchewancatch for the help!

@alalek
Copy link
Copy Markdown
Member

alalek commented Feb 15, 2020

I believe, this fix go into 3.4 too.

@vpisarev vpisarev changed the base branch from master to 3.4 February 15, 2020 09:11
@vpisarev
Copy link
Copy Markdown
Contributor Author

vpisarev commented Feb 15, 2020

@alalek, oops, sorry, I meant to submit PR to 3.4 branch. fixed!

@asmorkalov asmorkalov added the Hackathon https://opencv.org/opencv-hackathon-starts-next-week/ label Feb 17, 2020
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Thank you!

@alalek alalek merged commit d67a6c1 into opencv:3.4 Feb 17, 2020
@alalek alalek mentioned this pull request Feb 20, 2020
jshiwam pushed a commit to jshiwam/opencv that referenced this pull request Feb 22, 2020
fixed cv::moveWindow() on mac

* fixed cv::moveWindow() on mac (issue opencv#16343). Thanks to cwreynolds and saskatchewancatch for the help!

* fixed warnings about _x0 and _y0

* fixed warnings about _x0 and _y0
@vpisarev vpisarev deleted the fix_macos_move_window branch May 28, 2020 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: highgui-gui Hackathon https://opencv.org/opencv-hackathon-starts-next-week/ platform: ios/osx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

“y” argument to moveWindow() being ignored?

3 participants