This is the mail archive of the ecos-discuss@sources.redhat.com mailing list for the eCos 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: mqueue.inl


Gustav Kälvesten wrote:
> 
> Hi,
> 
> I think ecos/packages/kernel/current/include/mqueue.inl has a small error:
> 
> inline long
> Cyg_Mqueue::count()
> {
> ...
> }
> 
> should be like:
> 
> CYGPRI_KERNEL_SYNCH_MQUEUE_INLINE long
> Cyg_Mqueue::count()
> {
> ...
> }

I know that's what the other functions have, but that's because they are
quite large, whereas count() is small. The
CYGPRI_KERNEL_SYNCH_MQUEUE_INLINE macro is only present to prevent the test
inlining too much code, resulting in a massive binary.
 
> The current way gives me compilation errors for the kernel tests (mqueue1.cxx). Or maybe I'm missing something.

I suspect there must be some other reason. What's the error?

Jifl
-- 
Red Hat, Rustat House, Clifton Road, Cambridge, UK. Tel: +44 (1223) 271062
Maybe this world is another planet's Hell -Aldous Huxley || Opinions==mine


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