[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 03:33:32 -0600 |
User-agent: |
Heirloom mailx 12.4 7/29/08 |
Hi.
Please don't revert this change. dfa.c is *shared* code, and there's no
reason for gawk to put stuff needed ONLY by dfa.c into its config.h
file. (Gawk doesn't have a system.h - most of that stuff is in awk.h
at the momment. The rest is in custom.h which is used to override defaults
from configure, and not for portability.)
I understand that grep doesn't need it, but dfa serves more than
one master. I appreciate the grep team's accommodating gawk's needs.
Thanks,
Arnold
------------------
Paolo Bonzini <address@hidden> wrote:
> This partially reverts the following commits:
> - df6da5d40a47abbc6e3451cb9fab7a8c9ede12cc.
> - c40e7f7158109985b901ae67e5faa06160547471.
> - 0995fc11214ebbb1c99a4b6d675fe615b0ed8539.
>
> As far as grep is concerned, the #define *does* belong in gnulib. gawk
> can apply the fix in a common header file like grep's src/system.h.
> ---
> src/dfa.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/src/dfa.c b/src/dfa.c
> index b22fe97..a7f0056 100644
> --- a/src/dfa.c
> +++ b/src/dfa.c
> @@ -34,11 +34,7 @@
> #include <locale.h>
> #include <stdbool.h>
>
> -/* Gawk doesn't use Gnulib, so don't assume that setlocale and
> - static_assert are present. */
> -#ifndef LC_ALL
> -# define setlocale(category, locale) NULL
> -#endif
> +/* Gawk doesn't use Gnulib, so don't assume static_assert is present. */
> #ifndef static_assert
> # define static_assert(cond, diagnostic) \
> extern int (*foo (void)) [!!sizeof (struct { int foo: (cond) ? 8 : -1;
> })]
> --
> 1.9.0
>
>
>
>
> From address@hidden Tue Apr 1 03:19:14 2014
> X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on
> frenzy.freefriends.org
> X-Spam-Level:
> X-Spam-Status: No, score=-7.5 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_HI,
> RP_MATCHES_RCVD autolearn=ham version=3.3.1
> X-Envelope-From: address@hidden
> Return-Path: <address@hidden>
> X-Loop: address@hidden
> Subject: bug#17157: [PATCH 1/5] Partially revert "dfa: improve port to
> freestanding DJGPP"
> Resent-From: Paolo Bonzini <address@hidden>
> Original-Sender: "Debbugs-submit" <address@hidden>
> Resent-CC: address@hidden
> Resent-Date: Tue, 01 Apr 2014 09:19:03 +0000
> Resent-Message-ID: <address@hidden>
> Resent-Sender: address@hidden
> X-GNU-PR-Message: report 17157
> X-GNU-PR-Package: grep
> X-GNU-PR-Keywords: patch
> To: address@hidden
> X-Debbugs-Original-To: address@hidden
> From: Paolo Bonzini <address@hidden>
> Date: Tue, 1 Apr 2014 11:18:42 +0200
> X-Mailer: git-send-email 1.9.0
> X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address
> (bad octet value).
> X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x
> X-BeenThere: address@hidden
> X-Mailman-Version: 2.1.15
> Precedence: list
> X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x
> X-Received-From: 140.186.70.43
> Cc: address@hidden
> X-BeenThere: address@hidden
> List-Id: Bug reports for GNU grep <bug-grep.gnu.org>
> List-Unsubscribe: <https://lists.gnu.org/mailman/options/bug-grep>,
> <mailto:address@hidden>
> List-Archive: <http://lists.gnu.org/archive/html/bug-grep>
> List-Post: <mailto:address@hidden>
> List-Help: <mailto:address@hidden>
> List-Subscribe: <https://lists.gnu.org/mailman/listinfo/bug-grep>,
> <mailto:address@hidden>
> Errors-To: address@hidden
> Sender: address@hidden
> Status: R
>
> This partially reverts the following commits:
> - df6da5d40a47abbc6e3451cb9fab7a8c9ede12cc.
> - c40e7f7158109985b901ae67e5faa06160547471.
> - 0995fc11214ebbb1c99a4b6d675fe615b0ed8539.
>
> As far as grep is concerned, the #define *does* belong in gnulib. gawk
> can apply the fix in a common header file like grep's src/system.h.
> ---
> src/dfa.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/src/dfa.c b/src/dfa.c
> index b22fe97..a7f0056 100644
> --- a/src/dfa.c
> +++ b/src/dfa.c
> @@ -34,11 +34,7 @@
> #include <locale.h>
> #include <stdbool.h>
>
> -/* Gawk doesn't use Gnulib, so don't assume that setlocale and
> - static_assert are present. */
> -#ifndef LC_ALL
> -# define setlocale(category, locale) NULL
> -#endif
> +/* Gawk doesn't use Gnulib, so don't assume static_assert is present. */
> #ifndef static_assert
> # define static_assert(cond, diagnostic) \
> extern int (*foo (void)) [!!sizeof (struct { int foo: (cond) ? 8 : -1;
> })]
> --
> 1.9.0
>
>
>
>
>
- 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 <=
- 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, 2014/04/01
- 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