[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestandi
From: |
arnold |
Subject: |
bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP" |
Date: |
Tue, 01 Apr 2014 12:36:08 -0600 |
User-agent: |
Heirloom mailx 12.4 7/29/08 |
Hi Paul & Paolo,
Paolo Bonzini <address@hidden> wrote:
> Il 01/04/2014 16:43, Paul Eggert ha scritto:
> > Paolo Bonzini wrote:
> >> setlocale, static_assert etc. should be in a dfacompat.h file
> >
> > We already have a file that serves this function, called mbsupport.h. We
> > could rename it to dfaconfig.h (a better name than dfacompat.h, as it's
> > basically a separate config.h for dfa.c).
>
> Good idea.
>
> Paolo
Not really such a good idea. mbsupport.h is used in multiple places
in gawk to define if MBS support is availble. It needs to stay
that way. Since we're discussing keeping dfa simple in both its
environments, then it needs to be in a header that only dfa uses.
At the moment, the best candidate is xalloc.h, which differs somewhat
grep vs. gawk.
But here too, in reality that file is mainly for allocation stuff
and dfa portability doesn't really belong there.
I respectfully suggest dfaconfig.h (or dfacustom.h) with
#ifdef GAWK
#ifndef HAVE_SETLOCALE
#define selocale(x, y) NULL
#endif
#define static_assert(cond) ...
#endif
as the initial contents.
Much thanks,
Arnold
- bug#17156: [PATCH 0/5] Patches to apply, Paolo Bonzini, 2014/04/01
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Paolo Bonzini, 2014/04/01
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", arnold, 2014/04/01
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Paul Eggert, 2014/04/01
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Paolo Bonzini, 2014/04/01
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP",
arnold <=
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Paolo Bonzini, 2014/04/02
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", arnold, 2014/04/02
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Paolo Bonzini, 2014/04/02
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", arnold, 2014/04/02
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Paul Eggert, 2014/04/02
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Paul Eggert, 2014/04/03
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Jim Meyering, 2014/04/04
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Paul Eggert, 2014/04/05
- bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to freestanding DJGPP", Jim Meyering, 2014/04/06
bug#17156: [PATCH 3/5] grep: avoid to re-build a state built previously., Paolo Bonzini, 2014/04/01