Fix alternate random_double() implementation

hollasch · hollasch · commit 4516a32cdf2d · 2020-05-30T15:44:01.000-07:00
Resolves #621
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]

Original file line number	Diff line number	Diff line change
`@@ -1315,7 +1315,7 @@`
`1315`	`1315`	`inline double random_double() {`
`1316`	`1316`	`static std::uniform_real_distribution<double> distribution(0.0, 1.0);`
`1317`	`1317`	`static std::mt19937 generator;`
`1318`		`- return generator(distribution);`
	`1318`	`+ return distribution(generator);`
`1319`	`1319`	`}`
`1320`	`1320`	`~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~`
`1321`	`1321`	`[Listing [random-double-alt]: <kbd>[rtweekend.h]</kbd> random_double(), alternate implemenation]`