spectral added a comment.
In https://phab.mercurial-scm.org/D4370#69050, @yuja wrote: > Queued, thanks. > > > + def _loadchildrensetlazy(self, visit): > > + if not visit: > > + return None > > + if visit == 'all' or visit == 'this': > > + self._loadalllazy() > > + return None > > + > > + todel = [] > > + for k in visit: > > + kslash = k + '/' > > + ld = self._lazydirs.get(kslash) > > + if ld: > > + path, node, readsubtree = ld > > + self._dirs[kslash] = readsubtree(path, node) > > + todel.append(kslash) > > + for kslash in todel: > > + del self._lazydirs[kslash] > > Any reason not to use `self._loadlazy(kslash)`? Hmm, I feel like there was a previous version that iterated differently and meant I'd need a copy of the list to do this correctly.. in this version, no I don't think there's a reason to avoid it besides a potential minor speed difference; should I send a patch for that? REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D4370 To: spectral, #hg-reviewers Cc: yuja, mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel