]> Savannah Git Hosting - gnulib.git/commitdiff
dfa: tune via xzalloc
authorPaul Eggert <eggert@cs.ucla.edu>
Tue, 17 Dec 2019 21:07:15 +0000 (13:07 -0800)
committerPaul Eggert <eggert@cs.ucla.edu>
Wed, 18 Dec 2019 05:47:07 +0000 (21:47 -0800)
* lib/dfa.c (dfaoptimize): Prefer xzalloc to xmalloc + memset.

ChangeLog
lib/dfa.c

index 7925915f65c13b3561e683bd41cd21017e2ad2ed..ce3f8b2b7857068053bbe8254ec2f39a08b7b001 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2019-12-17  Paul Eggert  <eggert@cs.ucla.edu>
+
+       dfa: tune via xzalloc
+       * lib/dfa.c (dfaoptimize): Prefer xzalloc to xmalloc + memset.
+
 2019-12-17  Bruno Haible  <bruno@clisp.org>
 
        localcharset: Fix multithread-safety bug on Windows and OS/2.
index 00347f901471e86c7450990116454c03e0074784..dd2fa84fd9fcc277c7896fc134d8ea9dc7723acb 100644 (file)
--- a/lib/dfa.c
+++ b/lib/dfa.c
@@ -2473,12 +2473,7 @@ reorder_tokens (struct dfa *d)
 static void
 dfaoptimize (struct dfa *d)
 {
-  char *flags;
-  position_set merged0;
-  position_set *merged;
-
-  flags = xmalloc (d->tindex * sizeof *flags);
-  memset (flags, 0, d->tindex * sizeof *flags);
+  char *flags = xzalloc (d->tindex);
 
   for (idx_t i = 0; i < d->tindex; i++)
     {
@@ -2497,7 +2492,8 @@ dfaoptimize (struct dfa *d)
 
   flags[0] |= OPT_QUEUED;
 
-  merged = &merged0;
+  position_set merged0;
+  position_set *merged = &merged0;
   alloc_position_set (merged, d->nleaves);
 
   d->constraints = xnmalloc (d->tindex, sizeof *d->constraints);