potok/opencv
1
0
Fork 0
mirror of https://github.com/opencv/opencv.git synced 2026-07-21 19:33:03 +04:00
Code Packages Releases Wiki Activity

imgproc: remove redundant fabs() in fitEllipseDirect

Browse Source 
In fitEllipseDirect, `double det = fabs(cv::determinant(M))` applies
fabs() unnecessarily since the next line `if (fabs(det) > 1.0e-10)`
already takes the absolute value. Remove the outer fabs() to avoid
the redundant operation.
This commit is contained in:
gideok Kim
2026-03-02 21:31:57 +09:00
parent 7644636904
commit 6676c04d7c
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/imgproc/src/shapedescr.cpp
+1 -1
View File
@@ -750,7 +750,7 @@ cv::RotatedRect cv::fitEllipseDirect( InputArray _points )
M(2,1) = (DM(0,1) + (DM(0,3)*TM(0,1) + DM(0,4)*TM(1,1) + DM(0,5)*TM(2,1))/Ts)/2.;
M(2,2) = (DM(0,2) + (DM(0,3)*TM(0,2) + DM(0,4)*TM(1,2) + DM(0,5)*TM(2,2))/Ts)/2.;
double det = fabs(cv::determinant(M));
double det = cv::determinant(M);
if (fabs(det) > 1.0e-10)
break;
eps = (float)(s/(n*2)*1e-2);