diff options
author | Christopher Allan Webber <cwebber@dustycloud.org> | 2013-01-22 14:32:00 -0600 |
---|---|---|
committer | Christopher Allan Webber <cwebber@dustycloud.org> | 2013-01-22 14:32:00 -0600 |
commit | 0ac8317ddceee6d705c175e69c0825078246c7b9 (patch) | |
tree | 1838842f74e4d05fd39f42f1aa93dfbb88db9686 /mediagoblin/storage | |
parent | 36ae6bcbbb0fc3ab0dbbc8dcba0664b3d7c5096f (diff) | |
download | mediagoblin-0ac8317ddceee6d705c175e69c0825078246c7b9.tar.lz mediagoblin-0ac8317ddceee6d705c175e69c0825078246c7b9.tar.xz mediagoblin-0ac8317ddceee6d705c175e69c0825078246c7b9.zip |
Removing docstring bit about delete_file possibly deleting directories in the future
I agree that delete_dir as a separate operation is a better way to do
things, especially since there is a non-recursive deletion option that
will politely fail if the directory is not empty.
Diffstat (limited to 'mediagoblin/storage')
-rw-r--r-- | mediagoblin/storage/__init__.py | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/mediagoblin/storage/__init__.py b/mediagoblin/storage/__init__.py index 77e4212e..57218eb5 100644 --- a/mediagoblin/storage/__init__.py +++ b/mediagoblin/storage/__init__.py @@ -102,10 +102,6 @@ class StorageInterface(object): def delete_file(self, filepath): """ Delete or dereference the file (not directory) at filepath. - - TODO: is the below comment correct? AFAIK, we won't clean up empty directories... - This might need to delete directories, buckets, whatever, for - cleanliness. (Be sure to avoid race conditions on that though) """ # Subclasses should override this method. self.__raise_not_implemented() |