[ovs-dev] [=daemonpy 3/4] daemon.py: Don't shadow built-in 'file' variable.
Reid Price
reid at nicira.com
Sun Sep 18 02:12:52 UTC 2011
Didn't look at the entire file, but the patch seems obviously correct.
On Fri, Sep 16, 2011 at 6:28 PM, Ethan Jackson <ethan at nicira.com> wrote:
> Pychecker considers it bad style.
> ---
>
> The original version of this patch that I sent out broke the unit tests.
> Please review this version.
>
> ---
> python/ovs/daemon.py | 24 ++++++++++++------------
> 1 files changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/python/ovs/daemon.py b/python/ovs/daemon.py
> index a14be82..5b9b06a 100644
> --- a/python/ovs/daemon.py
> +++ b/python/ovs/daemon.py
> @@ -141,23 +141,23 @@ def _make_pidfile():
> # unlock the lock for us, and we don't want that.
> global file
>
> - file = open(tmpfile, "w")
> + file_handle = open(tmpfile, "w")
> except IOError, e:
> _fatal("%s: create failed (%s)" % (tmpfile, e.strerror))
>
> try:
> - s = os.fstat(file.fileno())
> + s = os.fstat(file_handle.fileno())
> except IOError, e:
> _fatal("%s: fstat failed (%s)" % (tmpfile, e.strerror))
>
> try:
> - file.write("%s\n" % pid)
> - file.flush()
> + file_handle.write("%s\n" % pid)
> + file_handle.flush()
> except OSError, e:
> _fatal("%s: write failed: %s" % (tmpfile, e.strerror))
>
> try:
> - fcntl.lockf(file, fcntl.LOCK_EX | fcntl.LOCK_NB)
> + fcntl.lockf(file_handle, fcntl.LOCK_EX | fcntl.LOCK_NB)
> except IOError, e:
> _fatal("%s: fcntl failed: %s" % (tmpfile, e.strerror))
>
> @@ -407,7 +407,7 @@ def __read_pidfile(pidfile, delete_if_stale):
> pass
>
> try:
> - file = open(pidfile, "r+")
> + file_handle = open(pidfile, "r+")
> except IOError, e:
> if e.errno == errno.ENOENT and delete_if_stale:
> return 0
> @@ -417,11 +417,11 @@ def __read_pidfile(pidfile, delete_if_stale):
> # Python fcntl doesn't directly support F_GETLK so we have to just try
> # to lock it.
> try:
> - fcntl.lockf(file, fcntl.LOCK_EX | fcntl.LOCK_NB)
> + fcntl.lockf(file_handle, fcntl.LOCK_EX | fcntl.LOCK_NB)
>
> # pidfile exists but wasn't locked by anyone. Now we have the
> lock.
> if not delete_if_stale:
> - file.close()
> + file_handle.close()
> logging.warning("%s: pid file is stale" % pidfile)
> return -errno.ESRCH
>
> @@ -429,7 +429,7 @@ def __read_pidfile(pidfile, delete_if_stale):
> try:
> raced = False
> s = os.stat(pidfile)
> - s2 = os.fstat(file.fileno())
> + s2 = os.fstat(file_handle.fileno())
> if s.st_ino != s2.st_ino or s.st_dev != s2.st_dev:
> raced = True
> except IOError:
> @@ -447,7 +447,7 @@ def __read_pidfile(pidfile, delete_if_stale):
> return -e.errno
> else:
> logging.debug("%s: deleted stale pidfile" % pidfile)
> - file.close()
> + file_handle.close()
> return 0
> except IOError, e:
> if e.errno not in [errno.EACCES, errno.EAGAIN]:
> @@ -457,7 +457,7 @@ def __read_pidfile(pidfile, delete_if_stale):
> # Someone else has the pidfile locked.
> try:
> try:
> - return int(file.readline())
> + return int(file_handle.readline())
> except IOError, e:
> logging.warning("%s: read: %s" % (pidfile, e.strerror))
> return -e.errno
> @@ -466,7 +466,7 @@ def __read_pidfile(pidfile, delete_if_stale):
> return -errno.EINVAL
> finally:
> try:
> - file.close()
> + file_handle.close()
> except IOError:
> pass
>
> --
> 1.7.6.1
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20110917/00801704/attachment-0003.html>
More information about the dev
mailing list