This is the mail archive of the newlib@sourceware.org mailing list for the newlib 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: Pre-processor mismatch between [libc/machine/arm] strlen.c and strlen-armv7.s - may lead in duplicate definition


On Aug 21 11:51, Nick Clifton wrote:
> Hi Corinna,
> 
> >Somebody from arm can comment on this, please?
> 
> Markus is correct - the parenthesis is needed.
> 
> Cheers
>   Nick
> 
> PS. A better patch would be to define the tests in one place - arm_asm.h -
> and then check the results in both strlen,c and strlen-armv7.S.  In that way
> there would be no need to ensure synchronisation between the sources.  But
> if that is going to be done, then a full cleanup of the code in the
> machine/arm directory would probably be in order.

Thanks, patch commited as is for now.


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: pgpBVwUldjtCd.pgp
Description: PGP signature


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