!X.Empty() should be used instead of X.Size()

[Echelon9]

As an example, the DS_WeightedGraph.h implementation of GetShortestPath() and GetSpanningTree() use the following template:

for (col=0; X.Size(); col++)
    path.Insert(X.Pop(), _FILE_AND_LINE_);

It was probably intended to use !X.Empty() instead.

Note, there is a potential performance impact here (unquantified) given this is designed to be a performant algorithm.