Fix alternate random_double() implementation

Resolves #621
RayTracing · May 30, 2020 · 4516a32 · 4516a32
1 parent 8b8827b
commit 4516a32
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,8 +11,9 @@ Change Log -- Ray Tracing in One Weekend
   - Fix: Missing closing parenthesis in listing 10 (#603)
   - Fix: Tiny improvements to the lambertian::scatter() development (#604)
   - Fix: Correct geometry type and unit vector method in `ray_color()`, listing 20 (#606)
-  - Fix: Listing 30, `mixture_pdf` needs `shared_ptr` arguments (#608)
+  - Fix: Listing 26: alternate `random_double()` switched `distribution`, `generator` (#621)
   - Fix: Listing 28, 30: `light_shape` should have default material, not `0` (#607)
+  - Fix: Listing 30: `mixture_pdf` needs `shared_ptr` arguments (#608)
 
 
 ----------------------------------------------------------------------------------------------------

diff --git a/books/RayTracingInOneWeekend.html b/books/RayTracingInOneWeekend.html
@@ -1315,7 +1315,7 @@
     inline double random_double() {
         static std::uniform_real_distribution<double> distribution(0.0, 1.0);
         static std::mt19937 generator;
-        return generator(distribution);
+        return distribution(generator);
     }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     [Listing [random-double-alt]: <kbd>[rtweekend.h]</kbd> random_double(), alternate implemenation]