Thanks Jeff. Well spotted. Applied in trunk. Your patches are always welcome!
Richard/
On Fri, Oct 23, 2009 at 4:43 AM, Jeff Donner <jeffrey.donner@...> wrote:
> In stage.hh,
>
> class std_point_int_t
> {
> ...
> bool operator<( const stg_point_int_t& other ) const
> { return ((x < other.x)  (y < other.y) ); }
> ...
>
> This isn't consistent  for 2 points A, B it works when
> A's coordinates are both larger than B's or viceversa,
> but when each has one coordinate bigger than the corresponding other's,
> A can be < B and simultaneously B < A.
>
> Eg, A is (1, 5), B is (5, 1):
>
> is A < B ?
> A.x < B.x, so, yes.
>
> is B < A ?
> B.x is not < A.x, but it's an 'or' so we get another chance,
> B.y < A.y so, B < A, also.
>
> The patch uses the x coordinate, unless A.x == B.x, when it uses y.
>
> 
> Come build with us! The BlackBerry(R) Developer Conference in SF, CA
> is the only developer event you need to attend this year. Jumpstart your
> developing skills, take BlackBerry mobile applications to market and stay
> ahead of the curve. Join us from November 9  12, 2009. Register now!
> http://p.sf.net/sfu/devconference
> _______________________________________________
> Playerstagedevelopers mailing list
> Playerstagedevelopers@...
> https://lists.sourceforge.net/lists/listinfo/playerstagedevelopers
>
>

Richard Vaughan
Autonomy Lab / Computing Science / Simon Fraser University
