D30: merge: Removed sorting in casefolding detection, for a slight performance win

alex_gaynor (Alex Gaynor) phabricator at mercurial-scm.org
Thu Aug 10 09:10:02 EDT 2017


alex_gaynor added inline comments.

INLINE COMMENTS

> quark wrote in merge.py:765
> I think this would be incorrect, since `foldprefix` is mutated and used in the loop.

I think you're right, removing the first `sorted` should be safe though.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D30

To: alex_gaynor, durin42, dsp
Cc: quark, krbullock, dsp, mercurial-devel


More information about the Mercurial-devel mailing list