Code smell: Two space indentation

Replaced 106806 occurances of ^( +)(.*$) with

The ruby community almost universally (i.e. everyone but Luke, Markus, and the other eleven people
who learned ruby in the 1900s) uses two-space indentation.

3 Examples:

    The code:
        end

        # Tell getopt which arguments are valid
        def test_get_getopt_args
            element = Setting.new :name => "foo", :desc => "anything", :settings => Puppet::Util::Settings.new
            assert_equal([["--foo", GetoptLong::REQUIRED_ARGUMENT]], element.getopt_args, "Did not produce appropriate getopt args")

    becomes:
        end

        # Tell getopt which arguments are valid
        def test_get_getopt_args
          element = Setting.new :name => "foo", :desc => "anything", :settings => Puppet::Util::Settings.new
          assert_equal([["--foo", GetoptLong::REQUIRED_ARGUMENT]], element.getopt_args, "Did not produce appropriate getopt args")

    The code:
            assert_equal(str, val)

            assert_instance_of(Float, result)

        end

        # Now test it with a passed object
    becomes:
          assert_equal(str, val)

          assert_instance_of(Float, result)

        end

        # Now test it with a passed object
    The code:
        end

        assert_nothing_raised do
            klass[:Yay] = "boo"
            klass["Cool"] = :yayness
        end

    becomes:
        end

        assert_nothing_raised do
          klass[:Yay] = "boo"
          klass["Cool"] = :yayness
        end

1 files changed, 50 insertions, 50 deletions

diff --git a/lib/puppet/parser/ast/caseopt.rb b/lib/puppet/parser/ast/caseopt.rb
index 6cf36f94c..4e296e82f 100644
--- a/lib/puppet/parser/ast/caseopt.rb
+++ b/lib/puppet/parser/ast/caseopt.rb
@@ -1,64 +1,64 @@
 require 'puppet/parser/ast/branch'
 
 class Puppet::Parser::AST
-    # Each individual option in a case statement.
-    class CaseOpt < AST::Branch
-        attr_accessor :value, :statements
+  # Each individual option in a case statement.
+  class CaseOpt < AST::Branch
+    attr_accessor :value, :statements
 
-        # CaseOpt is a bit special -- we just want the value first,
-        # so that CaseStatement can compare, and then it will selectively
-        # decide whether to fully evaluate this option
+    # CaseOpt is a bit special -- we just want the value first,
+    # so that CaseStatement can compare, and then it will selectively
+    # decide whether to fully evaluate this option
 
-        def each
-            [@value,@statements].each { |child| yield child }
-        end
+    def each
+      [@value,@statements].each { |child| yield child }
+    end
 
-        # Are we the default option?
-        def default?
-            # Cache the @default value.
-            return @default if defined?(@default)
+    # Are we the default option?
+    def default?
+      # Cache the @default value.
+      return @default if defined?(@default)
 
-            if @value.is_a?(AST::ASTArray)
-                @value.each { |subval|
-                    if subval.is_a?(AST::Default)
-                        @default = true
-                        break
-                    end
-                }
-            else
-                @default = true if @value.is_a?(AST::Default)
-            end
+      if @value.is_a?(AST::ASTArray)
+        @value.each { |subval|
+          if subval.is_a?(AST::Default)
+            @default = true
+            break
+          end
+        }
+      else
+        @default = true if @value.is_a?(AST::Default)
+      end
 
-            @default ||= false
+      @default ||= false
 
-            @default
-        end
+      @default
+    end
 
-        # You can specify a list of values; return each in turn.
-        def eachvalue(scope)
-            if @value.is_a?(AST::ASTArray)
-                @value.each { |subval|
-                    yield subval.safeevaluate(scope)
-                }
-            else
-                yield @value.safeevaluate(scope)
-            end
-        end
+    # You can specify a list of values; return each in turn.
+    def eachvalue(scope)
+      if @value.is_a?(AST::ASTArray)
+        @value.each { |subval|
+          yield subval.safeevaluate(scope)
+        }
+      else
+        yield @value.safeevaluate(scope)
+      end
+    end
 
-        def eachopt
-            if @value.is_a?(AST::ASTArray)
-                @value.each { |subval|
-                    yield subval
-                }
-            else
-                yield @value
-            end
-        end
+    def eachopt
+      if @value.is_a?(AST::ASTArray)
+        @value.each { |subval|
+          yield subval
+        }
+      else
+        yield @value
+      end
+    end
 
-        # Evaluate the actual statements; this only gets called if
-        # our option matched.
-        def evaluate(scope)
-            @statements.safeevaluate(scope)
-        end
+    # Evaluate the actual statements; this only gets called if
+    # our option matched.
+    def evaluate(scope)
+      @statements.safeevaluate(scope)
     end
+  end
 end