From 98be6836cf3fbc3be39bc3dde6459c596d40a827 2006-10-23 22:10:50
From: walter.doerwald
Date: 2006-10-23 22:10:50
Subject: [PATCH] Simplify ifile: Remove all methods that call the base
class method and propagate the return value to ifile.
This is now done by path itself now.

---

diff --git a/IPython/Extensions/ipipe.py b/IPython/Extensions/ipipe.py
index 2345b6c..e659d18 100644
--- a/IPython/Extensions/ipipe.py
+++ b/IPython/Extensions/ipipe.py
@@ -844,136 +844,6 @@ class ifile(path.path):
     file (or directory) object.
     """
 
-    def __add_(self, other):
-        return ifile(path._base(self) + other)
-
-    def __radd_(self, other):
-        return ifile(other + path._base(self))
-
-    def __div_(self, other):
-        return ifile(path.__div__(self, other))
-
-    def getcwd():
-        return ifile(path.path.getcwd())
-    getcwd.__doc__ = path.path.getcwd.__doc__
-    getcwd = staticmethod(getcwd)
-
-    def abspath(self):
-        return ifile(path.path.abspath(self))
-    abspath.__doc__ = path.path.abspath.__doc__
-
-    def normcase(self):
-        return ifile(path.path.normcase(self))
-    normcase.__doc__ = path.path.normcase.__doc__
-
-    def normpath(self):
-        return ifile(path.path.normpath(self))
-    normpath.__doc__ = path.path.normpath.__doc__
-
-    def realpath(self):
-        return ifile(path.path.realpath(self))
-    realpath.__doc__ = path.path.realpath.__doc__
-
-    def expanduser(self):
-        return ifile(path.path.expanduser(self))
-    expanduser.__doc__ = path.path.expanduser.__doc__
-
-    def expandvars(self):
-        return ifile(path.path.expandvars(self))
-    expandvars.__doc__ = path.path.expandvars.__doc__
-
-    def dirname(self):
-        return ifile(path.path.dirname(self))
-    dirname.__doc__ = path.path.dirname.__doc__
-
-    parent = property(dirname, None, None, path.path.parent.__doc__)
-
-    def splitpath(self):
-        (parent, child) = path.path.splitpath(self)
-        return (ifile(parent), child)
-    splitpath.__doc__ = path.path.splitpath.__doc__
-
-    def splitdrive(self):
-        (drive, rel) = path.path.splitdrive(self)
-        return (ifile(drive), rel)
-    splitdrive.__doc__ = path.path.splitdrive.__doc__
-
-    def splitext(self):
-        (filename, ext) = path.path.splitext(self)
-        return (ifile(filename), ext)
-    splitext.__doc__ = path.path.splitext.__doc__
-
-    if hasattr(path.path, "splitunc"):
-        def splitunc(self):
-            (unc, rest) = path.path.splitunc(self)
-            return (ifile(unc), rest)
-        splitunc.__doc__ = path.path.splitunc.__doc__
-
-        def _get_uncshare(self):
-            unc, r = os.path.splitunc(self)
-            return ifile(unc)
-
-        uncshare = property(
-            _get_uncshare, None, None,
-            """ The UNC mount point for this path.
-            This is empty for paths on local drives. """)
-
-    def joinpath(self, *args):
-        return ifile(path.path.joinpath(self, *args))
-    joinpath.__doc__ = path.path.joinpath.__doc__
-
-    def splitall(self):
-        return map(ifile, path.path.splitall(self))
-    splitall.__doc__ = path.path.splitall.__doc__
-
-    def relpath(self):
-        return ifile(path.path.relpath(self))
-    relpath.__doc__ = path.path.relpath.__doc__
-
-    def relpathto(self, dest):
-        return ifile(path.path.relpathto(self, dest))
-    relpathto.__doc__ = path.path.relpathto.__doc__
-
-    def listdir(self, pattern=None):
-        return [ifile(child) for child in path.path.listdir(self, pattern)]
-    listdir.__doc__ = path.path.listdir.__doc__
-
-    def dirs(self, pattern=None):
-        return [ifile(child) for child in path.path.dirs(self, pattern)]
-    dirs.__doc__ = path.path.dirs.__doc__
-
-    def files(self, pattern=None):
-        return [ifile(child) for child in path.path.files(self, pattern)]
-    files.__doc__ = path.path.files.__doc__
-
-    def walk(self, pattern=None):
-        for child in path.path.walk(self, pattern):
-            yield ifile(child)
-    walk.__doc__ = path.path.walk.__doc__
-
-    def walkdirs(self, pattern=None):
-        for child in path.path.walkdirs(self, pattern):
-            yield ifile(child)
-    walkdirs.__doc__ = path.path.walkdirs.__doc__
-
-    def walkfiles(self, pattern=None):
-        for child in path.path.walkfiles(self, pattern):
-            yield ifile(child)
-    walkfiles.__doc__ = path.path.walkfiles.__doc__
-
-    def glob(self, pattern):
-        return map(ifile, path.path.glob(self, pattern))
-    glob.__doc__ = path.path.glob.__doc__
-
-    if hasattr(os, 'readlink'):
-        def readlink(self):
-            return ifile(path.path.readlink(self))
-        readlink.__doc__ = path.path.readlink.__doc__
-
-        def readlinkabs(self):
-            return ifile(path.path.readlinkabs(self))
-        readlinkabs.__doc__ = path.path.readlinkabs.__doc__
-
     def getmode(self):
         return self.stat().st_mode
     mode = property(getmode, None, None, "Access mode")
diff --git a/doc/ChangeLog b/doc/ChangeLog
index af5a4f7..dbabc3b 100644
--- a/doc/ChangeLog
+++ b/doc/ChangeLog
@@ -1,3 +1,9 @@
+2006-10-23  Walter Doerwald  <walter@livinglogic.de>
+
+	* IPython/Extensions/ipipe.py (ifile): Remove all methods that
+	call the base class method and propagate the return value to
+	ifile. This is now done by path itself.
+
 2006-10-15  Fernando Perez  <Fernando.Perez@colorado.edu>
 
 	* IPython/ipapi.py (IPApi.__init__): Added new entry to public