Changes in / [f923967:82d88d5] in sasview

File:

• src/sas/sascalc/fit/expression.py (modified) (4 diffs)

src/sas/sascalc/fit/expression.py

@@ -188 +188 @@
 
     # Initialize dictionary with available functions
-    global_context = {}
-    global_context.update(math.__dict__)
-    global_context.update(dict(arcsin=math.asin,arccos=math.acos,
-                               arctan=math.atan,arctan2=math.atan2))
-    global_context.update(context)
-    global_context.update(parameters)
-    global_context['id'] = id
-    local_context = {}
+    globals = {}
+    globals.update(math.__dict__)
+    globals.update(dict(arcsin=math.asin,arccos=math.acos,
+                        arctan=math.atan,arctan2=math.atan2))
+    globals.update(context)
+    globals.update(parameters)
+    globals['id'] = id
+    locals = {}
 
     # Define the constraints function
@@ -210 +210 @@
 
     #print("Function: "+functiondef)
-    # CRUFT: python < 3.0;  doc builder isn't allowing the following exec
-    # https://stackoverflow.com/questions/4484872/why-doesnt-exec-work-in-a-function-with-a-subfunction/41368813#comment73790496_41368813
-    #exec(functiondef, global_context, local_context)
-    eval(compile(functiondef, '<string>', 'exec'), global_context, local_context)
-    retfn = local_context['eval_expressions']
+    exec(functiondef, globals, locals)
+    retfn = locals['eval_expressions']
 
     # Remove garbage added to globals by exec
-    global_context.pop('__doc__', None)
-    global_context.pop('__name__', None)
-    global_context.pop('__file__', None)
-    global_context.pop('__builtins__')
+    globals.pop('__doc__',None)
+    globals.pop('__name__',None)
+    globals.pop('__file__',None)
+    globals.pop('__builtins__')
     #print globals.keys()
 
@@ -235 +232 @@
 
     # Break pairs into left set and right set
-    # Note: pairs is array or list, so use "len(pairs) > 0" to check for empty.
-    left,right = [set(s) for s in zip(*pairs)] if len(pairs) > 0 else ([],[])
-    while len(pairs) > 0:
+    left,right = [set(s) for s in zip(*pairs)] if pairs != [] else ([],[])
+    while pairs != []:
         #print "within",pairs
         # Find which items only occur on the right
@@ -265 +261 @@
     satisfies the partial ordering given by the pairs in partial order.
     """
-    # Note: pairs is array or list, so use "len(pairs) > 0" to check for empty.
-    left,right = zip(*pairs) if len(pairs) > 0 else ([],[])
+    left,right = zip(*pairs) if pairs != [] else ([],[])
     items = set(left)
     n = order_dependencies(pairs)