From fabded984d3f25bba4f70d0946e67c2b4dcdfa9e Mon Sep 17 00:00:00 2001
From: Michael DeHaan <michael.dehaan@gmail.com>
Date: Sun, 15 Jul 2012 11:09:15 -0400
Subject: [PATCH] minor code cleanup

---
 lib/ansible/utils.py | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)

diff --git a/lib/ansible/utils.py b/lib/ansible/utils.py
index 4c85ff29aad..df3780cf37a 100644
--- a/lib/ansible/utils.py
+++ b/lib/ansible/utils.py
@@ -71,14 +71,7 @@ def write_tree_file(tree, hostname, buf):
 
 def is_failed(result):
     ''' is a given JSON result a failed result? '''
-    failed = False
-    rc = 0
-    if type(result) == dict:
-        failed = result.get('failed', 0)
-        rc     = result.get('rc', 0)
-    if rc != 0:
-        return True    
-    return failed
+    return ((result.get('rc', 0) != 0) or (result.get('failed', False) in [ True, 'True', 'true']))
 
 def prepare_writeable_dir(tree):
     ''' make sure a directory exists and is writeable '''
@@ -200,8 +193,7 @@ def _template(text, vars, setup_cache=None):
     ''' run a text buffer through the templating engine '''
     vars = vars.copy()
     vars['hostvars'] = setup_cache
-    text = varReplace(unicode(text), vars)
-    return text
+    return varReplace(unicode(text), vars)
 
 def template(text, vars, setup_cache=None):
     ''' run a text buffer through the templating engine 
@@ -265,14 +257,10 @@ def md5(filename):
     infile.close()
     return digest.hexdigest()
 
-
-
 ####################################################################
 # option handling code for /usr/bin/ansible and ansible-playbook 
 # below this line
 
-# FIXME: move to seperate file
-
 class SortedOptParser(optparse.OptionParser):
     '''Optparser which sorts the options by opt before outputting --help'''
     def format_help(self, formatter=None):