Re: [PATCH 1/9] dfa: x2nrealloc starting from a NULL pointer works

[Jim Meyering]

Paolo Bonzini wrote:
> * src/dfa.c (parse_bracket_exp): Do not MALLOC mbcset parts the first time
> they are encountered.  Initialize chars_al correctly.

Yes, that works fine.  Good!
ACK.

>  src/dfa.c |   13 +------------
>  1 files changed, 1 insertions(+), 12 deletions(-)
>
> diff --git a/src/dfa.c b/src/dfa.c
> index c7ba8db..52050e0 100644
> --- a/src/dfa.c
> +++ b/src/dfa.c
> @@ -798,7 +798,7 @@ parse_bracket_exp (void)
>    int chars_al, range_sts_al, range_ends_al, ch_classes_al,
>      equivs_al, coll_elems_al;
>
> -  chars_al = 1;
> +  chars_al = 0;
>    range_sts_al = range_ends_al = 0;
>    ch_classes_al = equivs_al = coll_elems_al = 0;
>    if (MB_CUR_MAX > 1)
> @@ -881,8 +881,6 @@ parse_bracket_exp (void)
>                        /* Store the character class as wctype_t.  */
>                        wctype_t wt = wctype (class);
>
> -                      if (ch_classes_al == 0)
> -                        MALLOC(work_mbc->ch_classes, ++ch_classes_al);
...