comparison mercurial/logcmdutil.py @ 45564:a717de1cb624

log: remove unused argument from _makerevset()

I want to make getrevs() reusable for other commands, but the dependency graph
of this getrevs() function is a mess. Let's minimize it first.

45563:142f0dcf90d0

    b'prune': (b'ancestors(%s)', b'not %lr'),
    b'user': (b'user(%s)', b'%lr'),
}


def _makerevset(repo, match, pats, slowpath, opts):
    """Return a revset string built from log options and file patterns"""
    opts = dict(opts)
    # follow or not follow?
    follow = opts.get(b'follow') or opts.get(b'follow_first')

    if filematcher is None:

        def filematcher(ctx):
            return match

    expr = _makerevset(repo, match, pats, slowpath, opts)
    if opts.get(b'graph'):
        if repo.ui.configbool(b'experimental', b'log.topo'):
            if not revs.istopo():
                revs = dagop.toposort(revs, repo.changelog.parentrevs)
                # TODO: try to iterate the set lazily

45564:a717de1cb624

    b'prune': (b'ancestors(%s)', b'not %lr'),
    b'user': (b'user(%s)', b'%lr'),
}


def _makerevset(repo, pats, slowpath, opts):
    """Return a revset string built from log options and file patterns"""
    opts = dict(opts)
    # follow or not follow?
    follow = opts.get(b'follow') or opts.get(b'follow_first')

    if filematcher is None:

        def filematcher(ctx):
            return match

    expr = _makerevset(repo, pats, slowpath, opts)
    if opts.get(b'graph'):
        if repo.ui.configbool(b'experimental', b'log.topo'):
            if not revs.istopo():
                revs = dagop.toposort(revs, repo.changelog.parentrevs)
                # TODO: try to iterate the set lazily