[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: remove.c patch needed to compile CVS cor3eutils with GCC 4.0.2
From: |
Jim Meyering |
Subject: |
Re: remove.c patch needed to compile CVS cor3eutils with GCC 4.0.2 |
Date: |
Tue, 22 Nov 2005 22:53:46 +0100 |
Paul Eggert <address@hidden> wrote:
> I didn't install this since I figured remove.c might still be
> mutating, and I'm not sure what the assert was supposed to check.
>
> Also, I assumed you wanted to use only declaration-after-statement
> C99, not 'for (declaration ...' C99, since only the former is
> commonly supported in pre-C99 compilers, but I thought I'd ask to make
> sure.
>
> 2005-11-22 Paul Eggert <address@hidden>
>
> * src/remove.c (AD_push): Remove assert that doesn't compile.
> (rm): Don't assume C99 for (int i = ..., since GCC 4.0.2 rejects
> it in default mode.
The latter part is fine.
I fixed the first part 40 minutes ago by leaving the assertion
there and restoring the decl/defn I mistakenly removed.