kevincox added inline comments. INLINE COMMENTS
> Alphare wrote in status.rs:76 > I didn't realize that `sort_unstable_by_key` needed to return a `T` and not a > `&T`, that forces a `clone()`. I'm not sure that matters, we'll go with the > shortest code and see if that shows up in profiling later. `sort_unstable_by_key` doesn't require the function to return a T at all. It just needs to return an `Ord` type. Note that `&Ord` is `Ord` automatically. https://doc.rust-lang.org/std/primitive.slice.html#method.sort_unstable_by_key REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7927/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7927 To: Alphare, #hg-reviewers, marmoute, kevincox Cc: marmoute, durin42, kevincox, mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel