Fix up some std.rand syntax #1161 (#1162)

tgschultz · andrewrk · commit 3e94347e6152 · 2018-06-27T12:30:15.000-04:00
* Fix old syntax in rand

Ziggurat somehow did not get updated to latest syntax

* Fix broken float casts

f32 float casts somehow not updated to latest syntax
diff --git a/std/rand/index.zig b/std/rand/index.zig
@@ -116,7 +116,7 @@ pub const Random = struct {
     pub fn floatNorm(r: *Random, comptime T: type) T {
         const value = ziggurat.next_f64(r, ziggurat.NormDist);
         switch (T) {
-            f32 => return f32(value),
+            f32 => return @floatCast(f32, value),
             f64 => return value,
             else => @compileError("unknown floating point type"),
         }
@@ -128,7 +128,7 @@ pub const Random = struct {
     pub fn floatExp(r: *Random, comptime T: type) T {
         const value = ziggurat.next_f64(r, ziggurat.ExpDist);
         switch (T) {
-            f32 => return f32(value),
+            f32 => return @floatCast(f32, value),
             f64 => return value,
             else => @compileError("unknown floating point type"),
         }
diff --git a/std/rand/ziggurat.zig b/std/rand/ziggurat.zig
@@ -84,12 +84,12 @@ fn ZigTableGen(
 
     for (tables.x[2..256]) |*entry, i| {
         const last = tables.x[2 + i - 1];
-        *entry = f_inv(v / last + f(last));
+        entry.* = f_inv(v / last + f(last));
     }
     tables.x[256] = 0;
 
     for (tables.f[0..]) |*entry, i| {
-        *entry = f(tables.x[i]);
+        entry.* = f(tables.x[i]);
     }
 
     return tables;
@@ -160,3 +160,7 @@ test "ziggurant exp dist sanity" {
         _ = prng.random.floatExp(f64);
     }
 }
+
+test "ziggurat table gen" {
+    const table = NormDist;
+}

Original file line number	Diff line number	Diff line change
`@@ -84,12 +84,12 @@ fn ZigTableGen(`
`84`	`84`
`85`	`85`	`for (tables.x[2..256]) \|*entry, i\| {`
`86`	`86`	`const last = tables.x[2 + i - 1];`
`87`		`- *entry = f_inv(v / last + f(last));`
	`87`	`+ entry.* = f_inv(v / last + f(last));`
`88`	`88`	`}`
`89`	`89`	`tables.x[256] = 0;`
`90`	`90`
`91`	`91`	`for (tables.f[0..]) \|*entry, i\| {`
`92`		`- *entry = f(tables.x[i]);`
	`92`	`+ entry.* = f(tables.x[i]);`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`return tables;`
`@@ -160,3 +160,7 @@ test "ziggurant exp dist sanity" {`
`160`	`160`	`_ = prng.random.floatExp(f64);`
`161`	`161`	`}`
`162`	`162`	`}`
	`163`	`+`
	`164`	`+test "ziggurat table gen" {`
	`165`	`+ const table = NormDist;`
	`166`	`+}`