From 5bb0bc3b511104c3b40c27819c8900fb9881eca4 Mon Sep 17 00:00:00 2001 From: Sayamindu Dasgupta Date: Tue, 02 Mar 2010 19:38:54 +0000 Subject: Us a try: block instead of checking if the file is writeable or not --- diff --git a/extensions/cpsection/power/model.py b/extensions/cpsection/power/model.py index c02984a..33ec905 100644 --- a/extensions/cpsection/power/model.py +++ b/extensions/cpsection/power/model.py @@ -58,12 +58,13 @@ def set_automatic_pm(enabled): if using_powerd(): # powerd - if not os.access(POWERD_INHIBIT_FLAG, os.W_OK): - _logger.debug('File %s is not writeable' % POWERD_INHIBIT_FLAG) - return 0 if enabled == 'off' or enabled == 0: - fd = open(POWERD_INHIBIT_FLAG, 'w') - fd.close() + try: + fd = open(POWERD_INHIBIT_FLAG, 'w') + except IOError: + _logger.debug('File %s is not writeable' % POWERD_INHIBIT_FLAG) + else: + fd.close() else: os.unlink(POWERD_INHIBIT_FLAG) return 0 -- cgit v0.9.1