On Jul 8, 2010, at 6:20 PM, David Blevins wrote: > The only way to fix that issue is to make the sorting done in References > stable, so it only makes changes in order necessary to enforce the references. > > Tricky bit of work indeed. I added a couple test cases to ReferencesTest > (_testNoReferences() and _testOrderedReferences()). If we can get those to > pass, we'll be in good shape.
Hey Ivan, Saw your patch for the stable reference sorting (OPENEJB-1309). Pretty great stuff! It's better than Dain's algorithm -- those that know Dain know how much of a compliment that is. Going to commit it right away. As we chatted on IRC, turning the Node into linked list with previous/next would be a super great compliment to the way the code is written. Looking forward to that patch, it's going to be excellent! Thanks so much for all the great work! -David
