commit 1263448196dec4a5b601dabeb56b5fb24c1d72fa
author benjaminwagner <benjaminwagner@google.com> Thu Mar 31 06:13:22 2016 -0700
committer Commit bot <commit-bot@chromium.org> Thu Mar 31 06:13:22 2016 -0700
tree 7ec519c1f0cf54a1f18a939fc399c21914a4ccba
parent cce19c821ce0a76886078c6df24fba57fd2f12de

Make SkRandom::next[US]Fixed1 private; update documentation for SkRandom::nextSScalar1.

SkRandom is unused in Chromium, Android, Mozilla, and Google3.

SkRandom::nextSScalar1 and LCGRandom::nextSScalar1 appear to me by inspection to potentially return -1, so I updated the documentation to match.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1846773002

Review URL: https://codereview.chromium.org/1846773002

src/effects/SkDiscretePathEffect.cpp

diff --git a/src/effects/SkDiscretePathEffect.cpp b/src/effects/SkDiscretePathEffect.cpp
index 5e68dc6..5af6ab4 100644
--- a/src/effects/SkDiscretePathEffect.cpp
+++ b/src/effects/SkDiscretePathEffect.cpp
@@ -47,7 +47,7 @@
     LCGRandom(uint32_t seed) : fSeed(seed) {}
 
     /** Return the next pseudo random number expressed as a SkScalar
-        in the range (-SK_Scalar1..SK_Scalar1).
+        in the range [-SK_Scalar1..SK_Scalar1).
     */
     SkScalar nextSScalar1() { return SkFixedToScalar(this->nextSFixed1()); }
 
@@ -61,7 +61,7 @@
     int32_t nextS() { return (int32_t)this->nextU(); }
 
     /** Return the next pseudo random number expressed as a signed SkFixed
-     in the range (-SK_Fixed1..SK_Fixed1).
+     in the range [-SK_Fixed1..SK_Fixed1).
      */
     SkFixed nextSFixed1() { return this->nextS() >> 15; }