[refactor] Remove indirection with call operator

I don't understand why it existed int he first place.
It adds anotherhasattr check, but as the attributes are already checked
in __init__, this should only have any meaning when attributes are
removed dynamically from the object after giving it to __init__.
This is not supported anymore!

Author: mxmlnkn

examples/context.py

@@ -9,7 +9,7 @@
 import mfusepy
 
 
-class Context(mfusepy.LoggingMixIn, mfusepy.Operations):
+class Context(mfusepy.Operations):
     'Example filesystem to demonstrate fuse_get_context()'
 
     def getattr(self, path, fh=None):

examples/loopback.py

@@ -5,69 +5,91 @@
 import logging
 import os
 import threading
-from os.path import realpath
 
 import mfusepy as fuse
 
 
-class Loopback(fuse.LoggingMixIn, fuse.Operations):
+def with_root_path(func):
+    def wrapper(self, path, *args, **kwargs):
+        return func(self, self.root + path, *args, **kwargs)
+
+    return wrapper
+
+
+def static_with_root_path(func):
+    def wrapper(self, path, *args, **kwargs):
+        return func(self.root + path, *args, **kwargs)
+
+    return wrapper
+
+
+class Loopback(fuse.Operations):
     def __init__(self, root):
-        self.root = realpath(root)
+        self.root = os.path.realpath(root)
         self.rwlock = threading.Lock()
 
-    def __call__(self, op, path, *args):
-        return super().__call__(op, self.root + path, *args)
-
+    @with_root_path
     def access(self, path, amode):
         if not os.access(path, amode):
             raise fuse.FuseOSError(errno.EACCES)
 
-    chmod = os.chmod
-    chown = os.chown
+    chmod = static_with_root_path(os.chmod)
+    chown = static_with_root_path(os.chown)
 
+    @with_root_path
     def create(self, path, mode, fi=None):
         return os.open(path, os.O_WRONLY | os.O_CREAT | os.O_TRUNC, mode)
 
+    @with_root_path
     def flush(self, path, fh):
         return os.fsync(fh)
 
+    @with_root_path
     def fsync(self, path, datasync, fh):
         if datasync != 0:
             return os.fdatasync(fh)
         return os.fsync(fh)
 
+    @fuse.log_callback
+    @with_root_path
     def getattr(self, path, fh=None):
         st = os.lstat(path)
         return {
             key: getattr(st, key)
             for key in ('st_atime', 'st_ctime', 'st_gid', 'st_mode', 'st_mtime', 'st_nlink', 'st_size', 'st_uid')
         }
 
+    @with_root_path
     def link(self, target, source):
         return os.link(self.root + source, target)
 
-    mkdir = os.mkdir
-    mknod = os.mknod
-    open = os.open
+    mkdir = static_with_root_path(os.mkdir)
+    mknod = static_with_root_path(os.mknod)
+    open = static_with_root_path(os.open)
 
+    @with_root_path
     def read(self, path, size, offset, fh):
         with self.rwlock:
             os.lseek(fh, offset, 0)
             return os.read(fh, size)
 
+    @with_root_path
     def readdir(self, path, fh):
         return ['.', '..', *os.listdir(path)]
 
-    readlink = os.readlink
+    readlink = static_with_root_path(os.readlink)
 
+    @with_root_path
     def release(self, path, fh):
         return os.close(fh)
 
+    @with_root_path
     def rename(self, old, new):
         return os.rename(old, self.root + new)
 
-    rmdir = os.rmdir
+    rmdir = static_with_root_path(os.rmdir)
 
+    @with_root_path
     def statfs(self, path):
         stv = os.statvfs(path)
         return {
@@ -86,16 +108,20 @@ def statfs(self, path):
             )
         }
 
+    @with_root_path
     def symlink(self, target, source):
         return os.symlink(source, target)
 
+    @with_root_path
     def truncate(self, path, length, fh=None):
         with open(path, 'r+') as f:
             f.truncate(length)
 
-    unlink = os.unlink
-    utimens = os.utime
+    unlink = static_with_root_path(os.unlink)
+    utimens = static_with_root_path(os.utime)
 
+    @fuse.log_callback
+    @with_root_path
     def write(self, path, data, offset, fh):
         with self.rwlock:
             os.lseek(fh, offset, 0)

examples/memory.py

@@ -11,7 +11,7 @@
 import mfusepy as fuse
 
 
-# LoggingMixIn should not be used anymore! This is only here for downwards compatibility tests.
+# Use log_callback instead of LoggingMixIn! This is only here for downwards compatibility tests.
 class Memory(fuse.LoggingMixIn, fuse.Operations):
     'Example memory filesystem. Supports only one level of files.'
 

examples/sftp.py

@@ -9,7 +9,7 @@
 import mfusepy
 
 
-class SFTP(mfusepy.LoggingMixIn, mfusepy.Operations):
+class SFTP(mfusepy.Operations):
     '''
     A simple SFTP filesystem. Requires paramiko: http://www.lag.net/paramiko/