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

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: Add note back. 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..81ce4c4a13ca420ef5ded5e953afc111b10666b0 100644
--- a/src/core/SkScaleToSides.h
+++ b/src/core/SkScaleToSides.h
@@ -45,10 +45,10 @@ 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
- // implemented in the ARM compiler.
+ // Note: nextafterf is a c99 call and should be std::nextafter, but this is not
+ // implemented in the GCC 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