* lib/dfa.c (dfaoptimize): Prefer xzalloc to xmalloc + memset.
---
 ChangeLog |  5 +++++
 lib/dfa.c | 10 +++-------
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 7925915f6..ce3f8b2b7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2019-12-17  Paul Eggert  <egg...@cs.ucla.edu>
+
+       dfa: tune via xzalloc
+       * lib/dfa.c (dfaoptimize): Prefer xzalloc to xmalloc + memset.
+
 2019-12-17  Bruno Haible  <br...@clisp.org>
 
        localcharset: Fix multithread-safety bug on Windows and OS/2.
diff --git a/lib/dfa.c b/lib/dfa.c
index 00347f901..dd2fa84fd 100644
--- 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);
-- 
2.17.1


Reply via email to