Gilles Chanteperdrix wrote:
> Philippe Gerum wrote:
>  > Gilles Chanteperdrix wrote:
>  > > Philippe Gerum wrote:
>  > >  > Gilles Chanteperdrix wrote:
>  > >  > > Hi Jan,
>  > >  > > 
>  > >  > > when compiling xenomai v2.4.x for ARM with gcc 4.2.1, I get plenty 
> of
>  > >  > > warnings like:
>  > >  > > xenomai-arm/kernel/xenomai/skins/posix/syscall.c:38:
>  > >  > > xenomai-arm/include/xenomai/rtdm/rtdm_driver.h:206: warning: 
> parameter has incomplete type
>  > >  > > 
>  > >  > > So it seems gcc 4.2.1 does not like the enum being used in function
>  > >  > > prototypes without being defines. So, could we move the enum 
> definition
>  > >  > > before the functions using it ?
>  > >  > > 
>  > >  > 
>  > >  > We can sort this out with a simple forward declaration.
>  > > 
>  > > That is the way it is done today, but gcc 4.2.1 seems not to like
>  > > it. Maybe recent C standards allow the compiler to pick the smallest type
>  > > for the enum as C++ does ?
>  > > 
>  > 
>  > Or any integral type the implementation sees fit actually. Well, yes, 
> maybe it
>  > is indeed bits of the C++ standard dripping on the C implementation of GCC,
>  > especially if forward decl of enums is not explicitly defined by the C 
> standard.
> 
> Actually it is gcc 3.4 which does not like the forward decl, not gcc 4.2

I'm fine with moving it if that helps to make all compilers happy. Just
ensure that it remains in the right doxygen group.

Jan

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Xenomai-core mailing list
Xenomai-core@gna.org
https://mail.gna.org/listinfo/xenomai-core

Reply via email to