[Lldb-commits] [PATCH] os.remove shouldn't fail, if file doesn't exist

Ed Maste emaste at freebsd.org
Fri Nov 21 17:04:34 PST 2014


Looks fine with a couple of minor comments.

================
Comment at: scripts/Python/buildSwigPython.py:431-433
@@ +430,5 @@
+"""
+Tries to remove given file. If file doesn't exist, the function will not throw.
+If an exception is thrown during remove, and errno is not ENOENT, then it will
+be rethrown. If errno is ENOENT, it will be ignored.
+"""
----------------
This comment seems overly verbose -- it's clear from reading the function that this is what happens.  What about something succinct like "Remove a file, ignoring error if it does not exist."


================
Comment at: scripts/Python/buildSwigPython.py:604
@@ -589,2 +603,3 @@
 
+
 #-----------------------------------------------------------------------------
----------------
random whitespace change

http://reviews.llvm.org/D6362






More information about the lldb-commits mailing list