Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix for error when using opencv3 #1358

Closed
wants to merge 1 commit into from

Conversation

iKrishneel
Copy link
Contributor

PR related to #1354 with correct indentation

@k-okada
Copy link
Member

k-okada commented Apr 26, 2016

may be #1377 solves your problem, if not please reopen

@k-okada k-okada closed this Apr 26, 2016
@iKrishneel
Copy link
Contributor Author

No it does not solve the problem.

This line fix https://github.com/jsk-ros-pkg/jsk_common/pull/1358/files#diff-07c8da014511eb62b187f89f14031c80R1701

solves the NULL Pointer error.
err

@k-okada k-okada reopened this Apr 27, 2016
@k-okada
Copy link
Member

k-okada commented Apr 27, 2016

I see, please rebase origin/master

@k-okada
Copy link
Member

k-okada commented May 1, 2016

thanks, created new PR with rebased origin/master -> #1387

@k-okada k-okada closed this May 1, 2016
@iKrishneel
Copy link
Contributor Author

iKrishneel commented May 2, 2016

@k-okada sensei,
I am sorry i overlooked it earlier.
Thanks for fixing it.

@k-okada
Copy link
Member

k-okada commented May 2, 2016

no, please check if latest origin/master work for you.

@iKrishneel
Copy link
Contributor Author

I tested the latest master and it works. 👍

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