This is the mail archive of the binutils@sources.redhat.com 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]

Re: [patch] [rfa] elf32-sh.c: Fix for relaxation testcases


On 11 Nov 2000, Alexandre Oliva wrote:
> On Nov 10, 2000, Nick Clifton <nickc@redhat.com> wrote:
> 
> > : 2000-10-29  Hans-Peter Nilsson  <hp@bitrange.com>
> > :
> > :        * elf32-sh.c (sh_elf_relocate_section): Add case for R_SH_NONE.
> 
> > Approved.
> 
> I like this one better.  It accomplishes exactly the same, but saves a
> little bit of work,

I wouldn't say that.  The R_SH_NONE case is rare enough that the test is
supposedly false most of the time.  Folding into the switch construct
probably saved more work (computewise).

> so I'm checking it in instead.  It seems that
> Bernd had forgotten to post it here :-(

Whatever.  It saves *me* work. :-)

brgds, H-P



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