[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/17] dfa: get rid of x*alloc
From: |
Jim Meyering |
Subject: |
Re: [PATCH 03/17] dfa: get rid of x*alloc |
Date: |
Fri, 12 Mar 2010 19:06:03 +0100 |
Paolo Bonzini wrote:
> * src/dfa.c: Include xalloc.h.
> (xmalloc, xrealloc, xcalloc): Remove.
> ---
> src/dfa.c | 35 ++---------------------------------
> src/dfa.h | 12 ------------
> 2 files changed, 2 insertions(+), 45 deletions(-)
>
> diff --git a/src/dfa.c b/src/dfa.c
...
The part above, described in the log, is fine.
However, the following ptr_t-removing change to dfa.h
doesn't belong in this change set.
> diff --git a/src/dfa.h b/src/dfa.h
> index f13a257..cff2b0b 100644
> --- a/src/dfa.h
> +++ b/src/dfa.h
> @@ -23,18 +23,6 @@
> In addition to clobbering modularity, we eat up valuable
> name space. */
>
> -#ifdef __STDC__
> -# ifndef _PTR_T
> -# define _PTR_T
> - typedef void * ptr_t;
> -# endif
> -#else
> -# ifndef _PTR_T
> -# define _PTR_T
> - typedef char * ptr_t;
> -# endif
> -#endif
> -
> /* Number of bits in an unsigned char. */
> #ifndef CHARBITS
> #define CHARBITS 8
- [PATCH 00/16] my last hefty patch drop, Paolo Bonzini, 2010/03/12
- [PATCH 01/17] kwset/system: remove ptr_t, Paolo Bonzini, 2010/03/12
- [PATCH 02/17] grep: cleanup one const cast, Paolo Bonzini, 2010/03/12
- [PATCH 03/17] dfa: get rid of x*alloc, Paolo Bonzini, 2010/03/12
- Re: [PATCH 03/17] dfa: get rid of x*alloc,
Jim Meyering <=
- [PATCH 04/17] dfa: remove CRANGE dead code, Paolo Bonzini, 2010/03/12
- [PATCH 05/17] dfa, grep: cleanup if-before-free and cast-of-argument-to-free, Paolo Bonzini, 2010/03/12
- [PATCH 07/17] syntax-check: enable makefile-TAB-only-indentation, Paolo Bonzini, 2010/03/12
- [PATCH 06/17] grep: fix error-message-uppercase, Paolo Bonzini, 2010/03/12
- [PATCH 09/17] syntax-check: enable space-tab, Paolo Bonzini, 2010/03/12