From 95fce0c619627c1501a071a6640b8bc3f7e6880f Mon Sep 17 00:00:00 2001 From: rfkelly0 Date: Sun, 7 Jun 2009 05:37:33 +0000 Subject: more notes on my API changes git-svn-id: http://pyfilesystem.googlecode.com/svn/branches/rfk-ideas@156 67cdc799-7952-0410-af00-57a81ceafa0f --- NOTES.txt | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) (limited to 'NOTES.txt') diff --git a/NOTES.txt b/NOTES.txt index 7e1df55..317b7d2 100644 --- a/NOTES.txt +++ b/NOTES.txt @@ -7,16 +7,19 @@ Minify the contained functions: - remove isabsolutepath(), it wasn't used Put error class definitions in separate submodule 'errors' - - deeper exception heirachy and less redundancy - (e.g. no more `raise UnsupportedError("UNSUPPORTED")`) + - less redundancy (e.g. no more `raise UnsupportedError("UNSUPPORTED")`) + - deeper exception heirachy (e.g. `ParentDirectoryMissingError`) +This is designed to allow me to be more optimistic; rather than checking all +preconditions before trying an action, I can just let it fail and branch on +the exception. Important for reducing the number of accesses to a remote FS. Remove the notion of hidden files from the base FS class. - - it's not the filesystem's business to decide what files I want to see - has lead to several bugs with copying/moving directories - - moved the logic into a separate wrapper class, currently - in fs.wrapfs.HideDotFiles + - it's not the filesystem's business to decide what files I want to see + - moved the logic into a separate wrapper class "HideDotFiles" Remove xattr support from base FS class, making it a separate interface. + - has lead to several bugs with copying/moving files and directories - now defined in fs.xattrs module - SimulateXAttr wrapper class contains the same logic - removexattr() -> delxattr() in line with python's get/set/del tradition @@ -31,5 +34,5 @@ Split up the test definitions a little: - use a separate FSTestCases mixin rather than subclassing TestOSFS - path helpers testcases in their own module - zipfs in its own module since it's different to all the others - - s3fs in its own module since it's very slow + - s3fs in its own module since it's very slow and costs money to test -- cgit v1.2.1