[boost] Re: Microsoft compiler(s) bug? Was: Formal Review: Typeof

From: Arkadiy Vertleyb (vertleyb_at_[hidden])
Date: 2005-05-21 19:37:55

"David Abrahams" <dave_at_[hidden]> wrote

> "Arkadiy Vertleyb" <vertleyb_at_[hidden]> writes:
>
> > "David Abrahams" <dave_at_[hidden]> wrote
> >
> >> I believe I have substantially simplified the implementation of typeof
> >> and fixed the vc8 compatibility problems.  See the enclosed
> >> typeof_impl.hpp
> >
> > Right.  That's exactly the simplification I meant in my previous post.
> > Unfortunately it means at() is instantiated BOOST_TYPEOF_LIMIT_SIZE
times,
> > whereas with more complicated expression it is instantiated only n
times,
> > where n is the actual size of the encoded vector.  I introduced this
> > complexity about half a year ago, and it cut the time of my tests more
than
> > in half...
>
> Okay, how about the one enclosed here, then?

Now you have both size() and size_<> instantiated LIMIT_SIZE times ;-)

> >> Also, on a stylistic note, the paucity of whitespace in
> >>
> >>    namespace boost{namespace type_of{
> >>
> >> seems a bit draconian (yes, I'm exercising my vocabulary today ;->).
> >
> > Well, there are two whitespaces :-)
> >
> > Why?  Does it make it unreadable?
>
> Just less readable than it might be.

I don't have strong feelings about it -- I'll add some whitespaces...

Regards,
Arkadiy