yuja added a comment.

  Can you add the following tests?
  
  - encoded filename differs from the original name (e.g. uppercase letter)
  - fncache disabled, but encodedstore is used
  
  >   def datafiles(self, matcher=None):
  >       for a, b, size in super(encodedstore, self).datafiles():
  > 
  > +            if not _matchtrackedpath(a, matcher):
  >  +                continue
  > 
  >   try:
  >       a = decodefilename(a)
  
  I'm pretty sure it's wrong to pass in an encoded filename to matcher.

REPOSITORY
  rHG Mercurial

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

To: pulkit, durin42, #hg-reviewers
Cc: yuja, mjpieters, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to