On Thu, Feb 28, 2013 at 10:14:46AM +0100, Hans de Goede wrote: > On 02/20/2013 06:09 PM, Hans de Goede wrote: > >On 02/20/2013 11:35 AM, Marc-André Lureau wrote: > >I think that should be: "(char*)p2 - (char*)p1" So that if the > >address of p2 is bigger they compare as p2 > p1, and qsort will > >keep the order of the array the same for monitors with identical > >coordinates. > > After further discussion, it turns out I'm wrong. So ACK for the patch > as is.
Having a summary of why this is right or wrong would be helful to have in the list archive if anyone (even you or Marc-Andre) wonder why this was done this way in a few months from now ;) Christophe
pgpTwMXpHWhXW.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel
