commit 35e70fb57f9e4093e759d0d35b6d9b8e7800a1a3
author Nicolas Pena <npm@chromium.org> Mon Mar 27 16:20:02 2017 -0400
committer Nicolás Peña <npm@chromium.org> Mon Mar 27 20:21:23 2017 +0000
tree e600bc1ed3d659b4079e58125b5b22e0c615f6ce
parent 76921113bf704588bd766b5b71d2df07a4ee0149

M58: Compare to epsilon instead of 0 in CFX_Matrix::SetReverse

Since we are going to divide i by 0, it is better to compare it to epsilon
and avoid wonkiness from division by something too close to 0.

BUG=chromium:702041

Change-Id: I8136d6063f8debd41cef37eaab7e4097b3f32f4b
Reviewed-on: https://pdfium-review.googlesource.com/3090
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
(cherry picked from commit fbd9ea1db2f1bb7fa006e7304a1202afc683c142)

Change-Id: I9b50a3940599faa91780c7a9209c3c2fb0ea152a
Reviewed-on: https://pdfium-review.googlesource.com/3235
Reviewed-by: Nicolás Peña <npm@chromium.org>

core/fxcrt/fx_basic_coords.cpp

diff --git a/core/fxcrt/fx_basic_coords.cpp b/core/fxcrt/fx_basic_coords.cpp
index cb5a010..b73dc6d 100644
--- a/core/fxcrt/fx_basic_coords.cpp
+++ b/core/fxcrt/fx_basic_coords.cpp
@@ -4,9 +4,8 @@
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
-#include <limits.h>
-
 #include <algorithm>
+#include <limits>
 
 #include "core/fxcrt/fx_coordinates.h"
 #include "core/fxcrt/fx_ext.h"
@@ -231,7 +230,7 @@
 
 void CFX_Matrix::SetReverse(const CFX_Matrix& m) {
   FX_FLOAT i = m.a * m.d - m.b * m.c;
-  if (FXSYS_fabs(i) == 0)
+  if (FXSYS_fabs(i) <= std::numeric_limits<float>::epsilon())
     return;
 
   FX_FLOAT j = -i;