quark added a comment.

  @durin42 The case-folding collision might be a small feature that Rust is 
meaningful.

INLINE COMMENTS

> merge.py:765
>      foldprefix = unfoldprefix = lastfull = ''
> -    for fold, f in sorted(foldmap.items()):
> +    for fold, f in foldmap.items():
>          if fold.startswith(foldprefix) and not f.startswith(unfoldprefix):

I think this would be incorrect, since `foldprefix` is mutated and used in the 
loop.

REPOSITORY
  rHG Mercurial

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

To: alex_gaynor, durin42, dsp
Cc: quark, krbullock, dsp, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to