openstack
/
swift
Code Issues Proposed changes

Get rid of strip_self 

It does the same thing as Python's staticmethod, so just use that instead.

Change-Id: Ib1974a6d315383583c36c440a503b39679c01443
This commit is contained in:
Tim Burke 2021-03-25 12:20:29 -07:00
parent 8cc1ef9255
commit ef44905b3d
1 changed files with 3 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
swift/obj/diskfile.py
View File

@ -647,15 +647,6 @@ def clear_auditor_status(devices, datadir, auditor_type="ALL"):
remove_file(auditor_status)
def strip_self(f):
"""
Wrapper to attach module level functions to base class.
"""
def wrapper(self, *args, **kwargs):
return f(*args, **kwargs)
return wrapper
class DiskFileRouter(object):
def __init__(self, *args, **kwargs):
@ -695,9 +686,9 @@ class BaseDiskFileManager(object):
diskfile_cls = None # must be set by subclasses
policy = None # must be set by subclasses
invalidate_hash = strip_self(invalidate_hash)
consolidate_hashes = strip_self(consolidate_hashes)
quarantine_renamer = strip_self(quarantine_renamer)
invalidate_hash = staticmethod(invalidate_hash)
consolidate_hashes = staticmethod(consolidate_hashes)
quarantine_renamer = staticmethod(quarantine_renamer)
def __init__(self, conf, logger):
self.logger = logger