Were there particular pain points before? The list of things to polish isn't short, but I don't mind reprioritizing things if it helps.

In D4312#77107, @idlsoft wrote:

I'm actually kinda looking forward to see if lfs works better with all the new changes.

> rbc = self.revbranchcache()
> + cl = self.changelog

Perhaps it has to be self.unfiltered().changelog because revbranchcache is
initialized with the unfiltered repo.

! wall 0.133497 comb 0.140000 user 0.130000 sys 0.010000 (best of 64)
! wall 0.139459 comb 0.150000 user 0.150000 sys 0.000000 (best of 57)

After
! wall 0.106518 comb 0.110000 user 0.100000 sys 0.010000 (best of 77)
! wall 0.106157 comb 0.110000 user 0.100000 sys 0.010000 (best of 83)

def makedb(path):
    """Construct a database handle for a database at path."""

• db = sqlite3.connect(path) + db = sqlite3.connect(pycompat.sysstr(path))

• • a/mercurial/debugcommands.py +++ b/mercurial/debugcommands.py @@ -1172,7 +1172,7 @@ if not util.safehasattr(index, 'stats'): raise error.Abort(_('debugindexstats only works with native code')) for k, v in sorted(index.stats().items()):
• ui.write('%s: %s\n' % (k, v)) + ui.write('%s: %d\n' % (pycompat.bytestr(k), v))

> + time seconds seconds name
> + 92.55 0.02 0.02 sleepext.py:7:sleep
> $ cat ../out | statprofran

I tried to glob this one by putting * at various places but I failed. Any help/suggestion on how to glob this will be helpful.

1. use pycompat.bytestr() as this patch does
2. use '%s' and add quotes in the output

How do we want to move forward with this?