Re: [Corelinux-develop] YAROGT part I

[Frank V. C. <fr...@co...>]

Christophe Prud'homme wrote:
> 
> > > * class is missing in Friend declaraion in Synchronized.hpp
> >
> > don't need it because it is foward declared in
> > DECLARE_CLASS(Synchronized) as per C++ standard.
> > Are you getting an error?
> Yes !

Ok, interesting, I'll re-review the C++ standard 1998(E)

> 
> > > * warning with corelinux::##tag (common.hpp) don't know yet how to fix it
> >
> > whats the warning?
> can't paste tag after token
> I''l send an email to the gcc dev mailing list

ok

> 
> > > BTW Frank why do you need to do this
> > >
> > > #if   !defined(__ITERATOR_HPP)
> > > #include <Iterator.hpp>
> > > #endif
> > >
> > > since it is done already in Iterator.hpp ?
> >
> > Please elucidate?
> 
> In any file you include, for example, Iterator.hpp
> 
> you do what I wrote a few lines above
> but in Iterator.hpp, you do
> 
> #if   !defined(__ITERATOR_HPP)
> #define  __ITERATOR_HPP
> 
> the code
> 
> #endif
> 
> so you do the same thing twice, you don't need to check Iterator.hpp multiple
> inclusion since it is handled by the header file itself !!

But in the calling module, if the compiler has already done it, it won't
have to incur the disk I/O of opening Iterator.hpp again.

> 
> --
>  Christophe Prud'homme                |
>  MIT, 77, Mass Ave, Rm 3-243          |
>  Cambridge MA 02139                   |  All laws are simulations of reality.
>  Tel (Office) : (00 1) (617) 253 0229 |  -- John C. Lilly
>  Fax (Office) : (00 1) (617) 258 8559 |
>  http://augustine.mit.edu/~prudhomm   |
>                           Following the hacker spirit
> 
> _______________________________________________
> Corelinux-develop mailing list
> Cor...@li...
> http://lists.sourceforge.net/mailman/listinfo/corelinux-develop

-- 
Frank V. Castellucci