Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(50)

Unified Diff: src/core/SkScaleToSides.h

Issue 1621453004: Use nextafter instead of nexttoward to avoid using long double calculation. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Switch to nextafterf to support arm. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/core/SkScaleToSides.h
diff --git a/src/core/SkScaleToSides.h b/src/core/SkScaleToSides.h
index c3fa172f672588d27c29e95436c0bc0320d05f3c..9e761ab36101e22c0f43824652304f54dc7c57cc 100644
--- a/src/core/SkScaleToSides.h
+++ b/src/core/SkScaleToSides.h
@@ -45,10 +45,8 @@ public:
// If newMaxRadius forces the total over the limit, then it needs to be
// reduced by one ULP to be less than limit - newMinRadius.
- // Note: nexttowardf is a c99 call and should be std::nexttoward, but this is not
mtklein 2016/01/22 17:44:32 The same note does still apply to nextafterf vs. s
herb_g 2016/01/22 17:52:34 Done.
- // implemented in the ARM compiler.
if (newMaxRadius + newMinRadius > limit) {
- newMaxRadius = nexttowardf(newMaxRadius, 0.0);
+ newMaxRadius = nextafterf(newMaxRadius, 0.0f);
}
*maxRadius = newMaxRadius;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698