<?xml version="1.0" encoding="UTF-8"?>
<commit>
  <added type="array"/>
  <modified type="array">
    <modified>
      <diff>@@ -50,10 +50,10 @@ module OpenWFE
   #   require 'openwfe/def'
   #
   #   class AnotherDefinition0 &lt; OpenWFE::ProcessDefinition
-  #    sequence do
-  #      participant :ref =&gt; &quot;toto&quot;
-  #      subprocess :ref =&gt; &quot;http://company.process.server/def0.rb&quot;
-  #    end
+  #     sequence do
+  #       participant :ref =&gt; &quot;toto&quot;
+  #       subprocess :ref =&gt; &quot;http://company.process.server/def0.rb&quot;
+  #     end
   #   end
   #
   # (but the engine parameter :remove_definitions_allowed must be set to true,
@@ -134,7 +134,6 @@ module OpenWFE
       end
     end
 
-    #
     # Takes care of cancelling the subprocess as well if any
     #
     def cancel</diff>
      <filename>lib/openwfe/expressions/fe_subprocess.rb</filename>
    </modified>
    <modified>
      <diff>@@ -31,7 +31,6 @@ module OpenWFE
   #
   module ValueMixin
 
-    #
     # Expressions that include the ValueMixin let it gather values and
     # then, in their reply() methods do the job with the values.
     # The gathering task is performed by the ValueMixin.
@@ -82,18 +81,18 @@ module OpenWFE
 
     private
 
-      def lookup (name_array, workitem)
-
-        name_array.each do |n|
+    def lookup (name_array, workitem)
 
-          v = lookup_string_attribute(n, workitem)
+      name_array.each do |n|
 
-          return v if v != nil
-            # covers v = false as well
-        end
+        v = lookup_string_attribute(n, workitem)
 
-        nil
+        return v if v != nil
+          # covers v == false as well
       end
+
+      nil
+    end
   end
 
 end</diff>
      <filename>lib/openwfe/expressions/value.rb</filename>
    </modified>
    <modified>
      <diff>@@ -21,10 +21,10 @@ class FlowTest84b &lt; Test::Unit::TestCase
   # TEST 0
 
   class Test0 &lt; OpenWFE::ProcessDefinition
-     sub0
-     define &quot;sub0&quot; do
-       toto
-     end
+    sub0
+    define &quot;sub0&quot; do
+      toto
+    end
   end
 
   def test_0</diff>
      <filename>old_test/ft_84b_subrepr.rb</filename>
    </modified>
    <modified>
      <diff>@@ -60,7 +60,7 @@ end
     # no description
     sequence do
     end
-    define &quot;sub0&quot; do
+    define :name =&gt; &quot;sub0&quot; do
     end
   end
 
@@ -70,7 +70,6 @@ end
     assert_nil OpenWFE::ExpressionTree.get_description(tree)
 
     tree = OpenWFE::DefParser.parse Test2b
-    #p tree
     assert_nil OpenWFE::ExpressionTree.get_description(tree)
   end
 end</diff>
      <filename>test/unit/ut_8_parser_description.rb</filename>
    </modified>
  </modified>
  <removed type="array"/>
  <parents type="array">
    <parent>
      <id>f7e16d003651193c3e282a312c191888f6706940</id>
    </parent>
  </parents>
  <author>
    <name>John Mettraux</name>
    <email>jmettraux@gmail.com</email>
  </author>
  <url>http://github.com/jmettraux/ruote/commit/e7c6652f5911959e5883ac608cb8cf538821c429</url>
  <id>e7c6652f5911959e5883ac608cb8cf538821c429</id>
  <committed-date>2009-05-19T23:43:47-07:00</committed-date>
  <authored-date>2009-05-19T23:43:47-07:00</authored-date>
  <message>define 'sub0' is no no; adapted test</message>
  <tree>b0179ea9f3a8b18af605615f7630128241ff4ec3</tree>
  <committer>
    <name>John Mettraux</name>
    <email>jmettraux@gmail.com</email>
  </committer>
</commit>
