Are you referring to this function in target.cpp? It looks fine to me.
Code:
/**
* Compare two targets.
* Compare the score of two targets for the sort function in C++.
* @param t1 the first Target
* @param t2 the second Target
* @returns (1, 0, or -1) for the scores of t1 > t2, t1 == t2, and t1 < t2
*/
int compareTargets(Target t1, Target t2)
{
if (t1.m_score > t2.m_score) return 1;
if (t1.m_score < t2.m_score) return -1;
return 0;
}