[REF] Use the contextlib.closing function instead of a couple of try/catch for

the release of the database cursor.

bzr revid: stw@openerp.com-20140109093258-vmr1a3gaxruo0psp

1 files changed, 4 insertions, 5 deletions

diff --git a/openerp/cli/server.py b/openerp/cli/server.py
index cf4d3855a84..ee2e90bccbd 100644
--- a/openerp/cli/server.py
+++ b/openerp/cli/server.py
@@ -85,10 +85,9 @@ def setup_pid_file():
     """
     config = openerp.tools.config
     if config['pidfile']:
-        fd = open(config['pidfile'], 'w')
-        pidtext = "%d" % (os.getpid())
-        fd.write(pidtext)
-        fd.close()
+        with open(config['pidfile'], 'w') as fd:
+            pidtext = "%d" % (os.getpid())
+            fd.write(pidtext)
 
 def preload_registry(dbname):
     """ Preload a registry, and start the cron."""
@@ -173,7 +172,7 @@ def main(args):
     if config['workers']:
         openerp.multi_process = True
 
-    # preload registryies, needed for -u --stop_after_init
+    # preload registries, needed for -u --stop_after_init
     rc = 0
     if config['db_name']:
         for dbname in config['db_name'].split(','):