[boost] Re: [range] update on ADL hooks

From: Thorsten Ottosen (nesotto_at_[hidden])
Date: 2005-03-24 05:56:05

"Thomas Witt" <witt_at_[hidden]> wrote in message 
news:d1tceq$m29$1_at_sea.gmane.org...
|
| Thorsten,
|
| Thorsten Ottosen wrote:
| > Dear all,
| >
| > I'm going to change the ADL lookup hook of range functions.
|
| Actually AFAICS you are changing a basic feature of the lib.
|
| >
| > instead of the irritating
| >
| > void foo()
| > {
| >     using namespace boost;
| >     begin(r);
| > }
|
| For the record I don't think it's irritating at all. It very clearly
| states what is going on. I can see it being cumbersome though.
|
| BTW shouldn't it be
|
|      using boost::begin;
|      begin(r);

yes!

its irritating especially for boost authors because it dosn't wok
portable; putting the code into boost::begin()
solves that once and for all.

| >
| > we simply now say
| >
| > void foo()
| > {
| >     boost::begin(r);
| > }
| >
| > and get full ADL lookup. This changes the extension protocol
| > to overiding these three functions:
|
| To me the key difference here is not the way hooks a provided, but that
| currently the user chooses to take advantage of ADL, while with the new
| scheme the library makes the choice for the user. I am unconvinced that
| that's actually better.
|
| > range_adl_begin()
| > range_adl_end()
| > range_adl_size()
|
| My prior remarks notwithstanding, I think the _adl part should go.

ok

-Thorsten