Boost logo

Boost Users :

Subject: Re: [Boost-users] [typeindex v3.0] Peer review begins Mon 21st ends Wed 30th
From: Antony Polukhin (antoshkka_at_[hidden])
Date: 2014-04-24 01:23:03


2014-04-23 17:33 GMT+04:00 Paul A. Bristow <pbristow_at_[hidden]>:
<...>

> > 2. What is your evaluation of the implementation?
> OK.
>
> Though I'm not enthusiastic about the name of namespace boost::typeind::
> but I
> don't have any much better ideas. Perhaps someone else has?
>
> Nor am I clear that this namespace bti helps the reader.
>
> "Through all the examples, we'll assume that the following namespace alias
> is in
> effect:
>
> namespace bti = boost::typeind;
>
> "
>
> IMO, the frequency of use of the namespace bti is probably so low that
> spelling
> it out is clearer, for example:
>
> boost::typeind::type_id<T>().pretty_name() // human readable
>

Agreed: boost::typeindex would be more user friendly. It suggests to the
reader which library to look for.
Other notes will be also addressed.

Thanks for the review!

-- 
Best regards,
Antony Polukhin


Boost-users list run by williamkempf at hotmail.com, kalb at libertysoft.com, bjorn.karlsson at readsoft.com, gregod at cs.rpi.edu, wekempf at cox.net