From 463696b1f555ab90d1c0b829a8164304e2e2f1ed Mon Sep 17 00:00:00 2001
From: Alexander Gordeyev <s0meuser@yandex.ru>
Date: Sat, 19 Dec 2015 17:29:47 +0300
Subject: [PATCH] refactor progress_from_line()

---
 gooey/gui/controller.py | 36 +++++++++++++++++++-----------------
 1 file changed, 19 insertions(+), 17 deletions(-)

diff --git a/gooey/gui/controller.py b/gooey/gui/controller.py
index c38b56c..c0cb18c 100644
--- a/gooey/gui/controller.py
+++ b/gooey/gui/controller.py
@@ -134,26 +134,28 @@ class Controller(object):
       return None
     progress_expr = self.build_spec['progress_expr']
     if progress_expr:
-      def safe_float(x):
-        try:
-          return float(x)
-        except ValueError:
-          return x
-      eval_locals = {k: safe_float(v) for k, v in match.groupdict().items()}
-      if "x" not in eval_locals:
-        eval_locals["x"] = [safe_float(x) for x in match.groups()]
-      try:
-        value = eval(progress_expr, {}, eval_locals)
-      except:
-        return None
+      return self._eval_progress(match, progress_expr)
     else:
+      return self._search_progress(match)
+
+  def _search_progress(self, match):
+    try:
+      return int(float(match.group(1)))
+    except:
+      return None
+
+  def _eval_progress(self, match, eval_expr):
+    def safe_float(x):
       try:
-        value = match.group(1)
-      except IndexError:
-        return None
+        return float(x)
+      except ValueError:
+        return x
+    _locals = {k: safe_float(v) for k, v in match.groupdict().items()}
+    if "x" not in _locals:
+      _locals["x"] = [safe_float(x) for x in match.groups()]
     try:
-      return int(value)
-    except ValueError:
+      return int(float(eval(eval_expr, {}, _locals)))
+    except:
       return None
 
   def process_result(self, process):