Closed mshabunin closed 4 months ago
For some reason ABI verification has failed on buildbot, it complains two function has been removed: https://pullrequest.opencv.org/buildbot/builders/precommit_linux64/builds/107321/steps/Compare%20ABI%20dumps/logs/report-html
The problem is related to the changes in this PR, but I'm not sure if this is a severe issue.
ABI check has been updated.
Changes:
CV_TERMCRIT_*
andCV_CMP_*
)drawContours
to use C++ API instead of callingcvDrawContours
+ test for filled contours with holes (case with non-filled contours is simpler and is covered in some other tests)Note:
There is one case where old drawContours behavior doesn't match the new one - when
contourIdx == -1
(means "draw all contours") andmaxLevel == 0
(means draw only selected contours, but not what is inside).From the docs:
Old behavior - only one first contour is drawn:
New behavior (also expected by the test) - all contours are drawn: