> > Sure, I did it this way so I could avoid changing of the header file and rebuilding > > of the whole source. The patch is definitely not meant to be applied as is. > > I've applied it as-is because properly running code always trumps theory and > discussion. If you'd like to clean it up anyway, of course go ahead.
There's nothing wrong with using an arbitrary field to break ties as long as the value of that field is stable. The order of equal elements is ultimately irrelevant, so any stable ordering solution is as good as any other.
|