[Differential] [Commented On] D30: merge: Removed sorting in casefolding detection, for a slight performance win

[alex_gaynor (Alex Gaynor)]

alex_gaynor added a comment.


  Is it important that the error be deterministic?
  
  I think we can improve the situation if you have exactly two collisions, by sorting `(f, foldmap[fold])` and `(lastfull, f)`, but I don't think it can easily be preserved for multiple collisions.

REPOSITORY
  rHG Mercurial

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

EMAIL PREFERENCES
  https://phab.mercurial-scm.org/settings/panel/emailpreferences/

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