[PATCH,V1 07/14] readelf/objdump: support for SFrame section

[Nick Clifton]

Hi Indu,

> This patch adds support for SFrame in readelf and objdump.
There appear to be a couple of problems with the readelf implementation of this feature:

   1. Just using the --sframe option on its own results in readelf saying that there is
      nothing to do:

       % ./readelf --sframe ../ld/tmpdir/sframe-bar.o
       readelf: Warning: Nothing to do.
       [...whole list of accepted command line options....]

   2. Not providing a section name gives a slightly off error message:

        % ./readelf --sframe= ../ld/tmpdir/sframe-bar.o
        Section '' has no data to dump.

      I would have expected something along the lines of "section name must be provided".

   3. Providing an incorrect section name does not result in any kind of error message,
      although the exit code is set to 1:

         % ./readelf --sframe=.eh_frame ../ld/tmpdir/sframe-bar.o
         % echo $?
         1

One other minor point - it would be good to mention support for this new feature in
the binutils/NEWS file.  (I know that you have a patch for gas/NEWS, I just think
that it should be mentioned here as well).

Cheers
   Nick