From 1263448196dec4a5b601dabeb56b5fb24c1d72fa Mon Sep 17 00:00:00 2001 From: benjaminwagner Date: Thu, 31 Mar 2016 06:13:22 -0700 Subject: 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 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/effects') diff --git a/src/effects/SkDiscretePathEffect.cpp b/src/effects/SkDiscretePathEffect.cpp index 5e68dc67ef..5af6ab4ceb 100644 --- a/src/effects/SkDiscretePathEffect.cpp +++ b/src/effects/SkDiscretePathEffect.cpp @@ -47,7 +47,7 @@ public: 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 @@ private: 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; } -- cgit v1.2.3