"m_MaximumCost higher than 0" is the most useless comment of them all. I can see the code, the thing is, I don't know why the comparison is being made. What is so special about m_MaximumCost being (less than) zero that it's being tested for here? What's the bigger idea?