]> git.draconx.ca Git - cdecl99.git/commitdiff
Unbreak random array declarators.
authorNick Bowler <nbowler@draconx.ca>
Sun, 18 Mar 2012 22:05:05 +0000 (18:05 -0400)
committerNick Bowler <nbowler@draconx.ca>
Sun, 18 Mar 2012 22:05:05 +0000 (18:05 -0400)
Passing really big values to gsl_rng_uniform_int is not supported by the
library.  Instead, add a function to generate any possible uintmax_t
value uniformly at random.

test/declgen.c

index af7dd8d6c12f625008f2e51e25cdad161ff2f4b2..d7bdbc1a40526c1262fa4b5dc908450b0f9dd739 100644 (file)
@@ -20,6 +20,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
+#include <limits.h>
 #include <errno.h>
 #include <assert.h>
 #include <stdbool.h>
@@ -268,6 +269,20 @@ gen_declspecs(struct gen_rng *rng, unsigned flags)
        return gen_randomize_specs(rng, s);
 }
 
+static uintmax_t gen_uintmax(struct gen_rng *rng)
+{
+       unsigned char tmp;
+       uintmax_t ret = 0;
+
+       for (size_t i = 0; i < sizeof ret; i++) {
+               tmp = gsl_rng_uniform_int(rng->rng, UCHAR_MAX+1);
+               ret <<= CHAR_BIT;
+               ret |= tmp;
+       }
+
+       return ret;
+}
+
 /*
  * Generate a random array declarator, selecting one of four possibilities
  * uniformly at random.
@@ -289,7 +304,7 @@ static void gen_array(struct gen_rng *rng, struct cdecl_declarator *d)
                d->u.array.length = 0;
                break;
        case 3:
-               d->u.array.length = gsl_rng_uniform_int(rng->rng, -1);
+               d->u.array.length = gen_uintmax(rng);
                break;
        default:
                assert(0);