public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* PATCH: Add CR16 support to readelf and replace NUM_ELEM with  ARRAY_SIZE
@ 2007-07-24 12:32 Nick Clifton
  2007-07-24 14:10 ` M R Swami Reddy
  0 siblings, 1 reply; 2+ messages in thread
From: Nick Clifton @ 2007-07-24 12:32 UTC (permalink / raw)
  To: binutils

[-- Attachment #1: Type: text/plain, Size: 722 bytes --]

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.


[-- Attachment #2: readelf.c.patch.bz2 --]
[-- Type: application/x-bzip2, Size: 3014 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: PATCH: Add CR16 support to readelf and replace NUM_ELEM  with ARRAY_SIZE
  2007-07-24 12:32 PATCH: Add CR16 support to readelf and replace NUM_ELEM with ARRAY_SIZE Nick Clifton
@ 2007-07-24 14:10 ` M R Swami Reddy
  0 siblings, 0 replies; 2+ messages in thread
From: M R Swami Reddy @ 2007-07-24 14:10 UTC (permalink / raw)
  To: Nick Clifton; +Cc: binutils


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.
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-07-24 12:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-07-24 12:32 PATCH: Add CR16 support to readelf and replace NUM_ELEM with ARRAY_SIZE Nick Clifton
2007-07-24 14:10 ` M R Swami Reddy

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).