Re: [Boost-users] [BGL] documentation problem

From: Jeremy Siek (jsiek_at_[hidden])
Date: 2004-04-04 12:41:11

Actually, I take that back, I think the implementation should be
changed to use get() instead of operator[] for w1.

On Apr 4, 2004, at 12:37 PM, Jeremy Siek wrote:

> Hi Val,
>
> That looks like a bug in the documentation for johnson's. It should 
> require
> Mutable Lvalue Property Map for the weight map.
>
> Cheers,
> Jeremy
>
> On Apr 4, 2004, at 9:42 AM, Val Samko wrote:
>> Hi,
>>
>> Is it required for a readable property map to have operator[]?
>> At least, that is not mentioned at
>> http://www.boost.org/libs/property_map/ReadablePropertyMap.html ,
>> so I assume, operator[] is not required.
>>
>> At http://www.boost.org/libs/graph/doc/johnson_all_pairs_shortest.html
>> it is documented that weight_map named parameter must be a model
>> of a readable property map.
>>
>> At johnson_all_pairs_shortest.hpp, line 101 we have
>> w[e2] = w1[*e];
>> which assumes that weight_map has operator [].
>>
>> Is the problem in the readable property map documentation,
>> or in the johnson_all_pairs_shortest, or I am doing something
>> wrong?
>>
>> --
>> Val Samko                    http://val.digiways.com
>>
>> _______________________________________________
>> Boost-users mailing list
>> Boost-users_at_[hidden]
>> http://lists.boost.org/mailman/listinfo.cgi/boost-users
>>
> _______________________________________________
> Jeremy Siek <jsiek_at_[hidden]>
> http://www.osl.iu.edu/~jsiek
> Ph.D. Student, Indiana University Bloomington
> Graduating in August 2004 and looking for work
> C++ Booster (http://www.boost.org)
> _______________________________________________
>
> _______________________________________________
> Boost-users mailing list
> Boost-users_at_[hidden]
> http://lists.boost.org/mailman/listinfo.cgi/boost-users
>
_______________________________________________
Jeremy Siek <jsiek_at_[hidden]>
http://www.osl.iu.edu/~jsiek
Ph.D. Student, Indiana University Bloomington
Graduating in August 2004 and looking for work
C++ Booster (http://www.boost.org)
_______________________________________________