This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
Re: Pedantic preprocessor
- From: Sergei Organov <s dot organov at javad dot com>
- To: Gary Thomas <gary at mlbassoc dot com>
- Cc: Sergei Gavrikov <w3sg at SoftHome dot net>, eCos patches list <ecos-patches at ecos dot sourceware dot org>
- Date: Tue, 16 Oct 2007 13:18:04 +0400
- Subject: Re: Pedantic preprocessor
- References: <20071013214906.GA9874@ubuntu> <47134EFE.9070602@mlbassoc.com>
Gary Thomas <gary@mlbassoc.com> writes:
> Sergei Gavrikov wrote:
[...]
>> ecosconfig new sam7ex256
>> ecosconfig tree
>> make -s
>> U CYGHWR_HAL_ARM_AT91, new inferred value AT91SAM7S
>> U CYGHWR_HAL_ARM_AT91_FIQ, new inferred value 1
>> U CYGHWR_HAL_ARM_AT91SAM7, new inferred value at91sam7x256
>> headers finished
>> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C
>> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C
>> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C
>> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C
>> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C
>> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C
>> build finished
>>
[...]
>> I like to see these 2 lines together. And if anybody knows how to fix
>> this issue in platform CDL, please, enlight me.
>>
> Your patch is the correct one - I've applied it. Next time, don't
> forget the ChangeLog entry.
Is there anybody here who is willing/capable to fix the rest of HALs the
same way, as every HAL in eCos suffers from the same problem (e.g., I
recently happened to do the same thing for SPARC HAL)?
--
Sergei.