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 tautologies in calibfilter.cpp which cause a build failure when using -Werror=address with clang-3.5 #3488

Merged
merged 1 commit into from
Dec 6, 2014

Conversation

jeremyhu
Copy link
Contributor

@jeremyhu jeremyhu commented Dec 5, 2014

http://code.opencv.org/issues/4048

calibfilter.cpp:98:9: error: comparison of array 'this->latestPoints' not equal to a null pointer is always true [-Werror,-Wtautological-pointer-compare]
if (latestPoints != __null)
^~~~~~~~~~ ~~~~

calibfilter.cpp:526:9: error: address of array 'this->latestCounts' will always evaluate to 'true' [-Werror,-Wpointer-bool-conversion]
if( latestCounts )
~ ^~~~~~~~~~~
2 errors generated.

Signed-off-by: Jeremy Huddleston Sequoia jeremyhu@apple.com

…sing -Werror=address with clang-3.5

http://code.opencv.org/issues/4048

Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>
@ilya-lavrenov ilya-lavrenov self-assigned this Dec 5, 2014
@ilya-lavrenov
Copy link
Contributor

👍

@opencv-pushbot opencv-pushbot merged commit 35f96d6 into opencv:2.4 Dec 6, 2014
vpisarev added a commit that referenced this pull request Dec 6, 2014
uqs pushed a commit to freebsd/freebsd-ports that referenced this pull request Dec 16, 2014
Obtained from:	opencv/opencv#3488
PR:		ports/195727


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@374818 35697150-7ecd-e111-bb59-0022644237b5
uqs pushed a commit to freebsd/freebsd-ports that referenced this pull request Dec 16, 2014
Obtained from:	opencv/opencv#3488
PR:		ports/195727
@nahidnasiri
Copy link

Finally I could solve my problem, thanks a lotttttt

svmhdvn pushed a commit to svmhdvn/freebsd-ports that referenced this pull request Jan 10, 2024
Obtained from:	opencv/opencv#3488
PR:		ports/195727
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.

None yet

4 participants