> 49 - if ((*it)->GetCenter().y + - _icon_size/ 2 < GetGeometry().y)
> 50 + if ((*it)->GetCenter().y - _icon_size / 2 < GetGeometry().y)
> 51 + {
>
> That should test for > 0. In the diff it initially looked like you were
> changing a + to a - :)
What do you mean? You'd prefer to move GetGeometry().y to the left side?
The old code contained a copy&paste typo because there was a "+ -" operator that is just a "-" in fact.
> 49 - if ((*it)- >GetCenter( ).y + - _icon_size/ 2 < GetGeometry().y) >GetCenter( ).y - _icon_size / 2 < GetGeometry().y)
> 50 + if ((*it)-
> 51 + {
>
> That should test for > 0. In the diff it initially looked like you were
> changing a + to a - :)
What do you mean? You'd prefer to move GetGeometry().y to the left side?
The old code contained a copy&paste typo because there was a "+ -" operator that is just a "-" in fact.