opencv fails to build with latest llvm |
||||||
Issue description
OpenCV build has several errors because of pointer comparison with 0 using > operator. It should be changed to !=
OpenCV-2.3.0/modules/core/src/persistence.cpp:3437:36:
error: ordered comparison between pointer and zero ('const char *' and 'int')
opencv-2.3.0-r5: if( rows < 0 || cols < 0 || dt < 0 )
,
Nov 15 2016
,
Nov 15 2016
,
Nov 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/0ff9ba13a989c73fc887c3426d39e4db05bfbce4 commit 0ff9ba13a989c73fc887c3426d39e4db05bfbce4 Author: Manoj Gupta <manojgupta@google.com> Date: Mon Nov 14 23:42:55 2016 Fix opencv ebuild to work with latest llvm. Fix clang error: ordered comparison between pointer and zero ('const char *' and 'int') The warning was already fixed upstream. BUG= chromium:665229 TEST=no more error when building Change-Id: I4cb1b6e1622872822442f99e191ecaf23eebf65c Reviewed-on: https://chromium-review.googlesource.com/411322 Commit-Ready: Manoj Gupta <manojgupta@chromium.org> Tested-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> [add] https://crrev.com/0ff9ba13a989c73fc887c3426d39e4db05bfbce4/media-libs/opencv/files/opencv-2.3.0-clang-error.patch [rename] https://crrev.com/0ff9ba13a989c73fc887c3426d39e4db05bfbce4/media-libs/opencv/opencv-2.3.0-r6.ebuild
,
Nov 16 2016
,
Mar 4 2017
,
Mar 17 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by manojgupta@chromium.org
, Nov 15 2016Status: Assigned (was: Untriaged)