Skip to content
Browse files

Adding comment about whether to return the destination file or not on…

… file copies.
  • Loading branch information...
1 parent ba46793 commit 7f7f08ed6228922f86fb87d90ffa598cbfd093f5 @robhudson committed
Showing with 4 additions and 0 deletions.
  1. +4 −0 dumpy/dumper.py
View
4 dumpy/dumper.py
@@ -303,6 +303,10 @@ def process(self, file):
shutil.copy(file.name, new_file_name)
new_file = open(new_file_name)
+ # TODO:
+ # This should probably not return the new file but return the original.
+ # In that way, any copy post processing results in a dead end but the
+ # original file can still be used in more post processing.
file.close()
return new_file

0 comments on commit 7f7f08e

Please sign in to comment.
Something went wrong with that request. Please try again.