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

From: Thomas Witt (witt_at_[hidden])
Date: 2005-03-23 22:33:48

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);

> 
> 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.

Thomas

-- 
Thomas Witt
witt_at_[hidden]