[debbugs-tracker] bug#17715: closed ([PATCH] dfa: build struct dfamust on demand)

[GNU bug Tracking System]

Your message dated Sat, 4 Jul 2015 17:17:42 -0700
with message-id <address@hidden>
and subject line Re: bug#17715: [PATCH] dfa: build struct dfamust on demand
has caused the debbugs.gnu.org bug report #17715,
regarding [PATCH] dfa: build struct dfamust on demand
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
17715: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=17715
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: [PATCH] dfa: build struct dfamust on demand Date: Fri, 06 Jun 2014 22:36:31 +0900

If we don't use KWset, struct dfamust doesn't have to build.  This patch
make a change that it's built on demand.

0001-dfa-build-struct-dfamust-on-demand.patch
Description: Text document

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#17715: [PATCH] dfa: build struct dfamust on demand
>
>
>
>
> Date: 
>
> Sat, 4 Jul 2015 17:17:42 -0700
>
>
>
>
> On Sat, Jul 4, 2015 at 2:51 PM, Jim Meyering <address@hidden> wrote:
> > On Sat, Jul 4, 2015 at 1:43 PM, Paul Eggert <address@hidden> wrote:
> >> Thanks for looking into this.
> >>
> >> One little thing I noticed while reading that patch.  dfaalloc could be
> >> declared with _GL_ATTRIBUTE_MALLOC; this is true both before and after the
> >> patch is applied.  I don't know if it's worth bothering to declaring it that
> >> way, though, as I doubt it'd improve the quality of the generated code.
> >> Plus, _GL_ATTRIBUTE_MALLOC isn't visible in dfa.h now.
> >
> > Good idea. I prefer to add it, on principle.
> > To permit that change, I'll move the inclusion of xalloc.h --
> > the header that defines it -- from dfa.c to dfa.h.
>
> Pushed with that change.
>
>
> --- End Message ---