This is an archive of the discontinued Mercurial Phabricator instance.

Differential D8994

localrepo: use functools.wraps() in unfilteredmethod decorator
ClosedPublic

Authored by durin42 on Sep 7 2020, 5:31 PM.

Details

Reviewers
pulkit
Group Reviewers
hg-reviewers
Commits
rHG4111954cf86d: localrepo: use functools.wraps() in unfilteredmethod decorator
rHG98baadff2b1c: localrepo: use functools.wraps() in unfilteredmethod decorator
Summary

This makes it easier to figure out what function you're holding on to
when doing printf-style debugging.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Revision Contents
Changeset List

PathPackages
Mmercurial/localrepo.py (2 lines)

Diff 22569

mercurial/localrepo.py

# localrepo.py - read/write repository class for mercurial # localrepo.py - read/write repository class for mercurial
# #
# Copyright 2005-2007 Matt Mackall <mpm@selenic.com> # Copyright 2005-2007 Matt Mackall <mpm@selenic.com>
# #
# This software may be used and distributed according to the terms of the # This software may be used and distributed according to the terms of the
# GNU General Public License version 2 or any later version. # GNU General Public License version 2 or any later version.
from __future__ import absolute_import from __future__ import absolute_import
import errno import errno
import functools
import os import os
import random import random
import sys import sys
import time import time
import weakref import weakref
from .i18n import _ from .i18n import _
from .node import ( from .node import (
def hasunfilteredcache(repo, name): def hasunfilteredcache(repo, name):
"""check if a repo has an unfilteredpropertycache value for <name>""" """check if a repo has an unfilteredpropertycache value for <name>"""
return name in vars(repo.unfiltered()) return name in vars(repo.unfiltered())
def unfilteredmethod(orig): def unfilteredmethod(orig):
"""decorate method that always need to be run on unfiltered version""" """decorate method that always need to be run on unfiltered version"""
@functools.wraps(orig)
def wrapper(repo, *args, **kwargs): def wrapper(repo, *args, **kwargs):
return orig(repo.unfiltered(), *args, **kwargs) return orig(repo.unfiltered(), *args, **kwargs)
return wrapper return wrapper
moderncaps = { moderncaps = {
b'lookup', b'lookup',