This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: PATCH: Add CR16 support to readelf and replace NUM_ELEM with ARRAY_SIZE

From: "M R Swami Reddy" <MR dot Swami dot Reddy at nsc dot com>
To: "Nick Clifton" <nickc at redhat dot com>
Cc: binutils at sourceware dot org
Date: Tue, 24 Jul 2007 18:15:40 +0530
Subject: Re: PATCH: Add CR16 support to readelf and replace NUM_ELEM with ARRAY_SIZE
References: <m3odi2nfxe.fsf@redhat.com>

Hi Nick,

Thanks a lot for adding CR16 support in readelf and the CR16 support
patch is fine.

Thanks,
Swami

Nick Clifton wrote:

Hi Guys,

  I am being a bit naughty here by combining three patches to readelf
  into one submission...

  I am applying the attached patch to fix a couple of minor things
  with readelf.  The first was that it did not recognize the newly
  added CR16 target.  The second was that there is no need for the
  NUM_ELEM() macro since a standard alternative version - ARRAY_SIZE
  - is defined in libiberty.h.  Finally the third fix is to correct a
  few formatting errors in recently added code.

Cheers
  Nick

binutils/ChangeLog
2007-07-24  Nick Clifton  <nickc@redhat.com>

	* readelf.c (NUM_ELEM): Remove redundant macro; replace references
	to it with references to ARRAY_SIZE.
	Add support for CR16.
	Fix formatting.

References:
- PATCH: Add CR16 support to readelf and replace NUM_ELEM with ARRAY_SIZE
  - From: Nick Clifton

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]