tetris: Use arc4random_uniform instead of modulo for better randomness

diff --git a/tetris/tetris.c b/tetris/tetris.c
index 5a41c158347b2a9780c3d0a99bbc203048b027a6..286e7c4550190f88c4097eb59018f8c0a4346295 100644 (file)
--- a/tetris/tetris.c
+++ b/tetris/tetris.c
@@ -1,4 +1,4 @@
-/*     $NetBSD: tetris.c,v 1.32 2016/03/03 21:38:55 nat Exp $  */
+/*     $NetBSD: tetris.c,v 1.33 2020/07/21 02:42:05 nia Exp $  */
 
 /*-
  * Copyright (c) 1992, 1993
@@ -207,7 +207,6 @@ main(int argc, char *argv[])
        scr_init();
        setup_board();
 
-       srandom(getpid());
        scr_set();
 
        pos = A_FIRST*B_COLS + (B_COLS/2)-1;

diff --git a/tetris/tetris.h b/tetris/tetris.h
index 7d18a2d1790c3d52c248ca0f629901a239a5d390..8e1479f198ca603125efb9302fa1a323c2fec7af 100644 (file)
--- a/tetris/tetris.h
+++ b/tetris/tetris.h
@@ -1,4 +1,4 @@
-/*     $NetBSD: tetris.h,v 1.15 2016/03/03 21:38:55 nat Exp $  */
+/*     $NetBSD: tetris.h,v 1.16 2020/07/21 02:42:05 nia Exp $  */
 
 /*-
  * Copyright (c) 1992, 1993
@@ -130,7 +130,7 @@ struct shape {
 };
 
 extern const struct shape shapes[];
-#define        randshape() (&shapes[random() % 7])
+#define        randshape() (&shapes[arc4random_uniform(7)])
 
 extern const struct shape *nextshape;