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@lists.boost.org http://lists.boost.org/mailman/listinfo.cgi/boost-users

_______________________________________________ Jeremy Siek 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@lists.boost.org http://lists.boost.org/mailman/listinfo.cgi/boost-users

_______________________________________________ Jeremy Siek 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) _______________________________________________