Code smell: Avoid needless decorations

* Replaced 704 occurances of (.*)\b([a-z_]+)\(\) with \1\2

  3 Examples:

      The code:
          ctx = OpenSSL::SSL::SSLContext.new()
      becomes:
          ctx = OpenSSL::SSL::SSLContext.new
      The code:
          skip()
      becomes:
          skip
      The code:
          path = tempfile()
      becomes:
          path = tempfile

* Replaced 31 occurances of ^( *)end *#.* with \1end

  3 Examples:

      The code:

      becomes:

      The code:
          end # Dir.foreach
      becomes:
          end
      The code:
          end # def
      becomes:
          end

2 files changed, 5 insertions, 5 deletions

diff --git a/lib/puppet/parser/functions/shellquote.rb b/lib/puppet/parser/functions/shellquote.rb
index 96feaa1ee..888b9769d 100644
--- a/lib/puppet/parser/functions/shellquote.rb
+++ b/lib/puppet/parser/functions/shellquote.rb
@@ -10,7 +10,7 @@ module Puppet::Parser::Functions
         with spaces.  If an argument is an array, the elements of that
         array is interpolated within the rest of the arguments; this makes
         it possible to have an array of arguments and pass that array to
-        shellquote() instead of having to specify each argument
+        shellquote instead of having to specify each argument
         individually in the call.
         ") \
     do |args|
@@ -25,9 +25,9 @@ module Puppet::Parser::Functions
                 result << ("'" + word + "'")
             else
                 r = '"'
-                word.each_byte() do |c|
+                word.each_byte do |c|
                     r += "\\" if Dangerous.include?(c)
-                    r += c.chr()
+                    r += c.chr
                 end
                 r += '"'
                 result << r
diff --git a/lib/puppet/parser/functions/sprintf.rb b/lib/puppet/parser/functions/sprintf.rb
index 4a3916c89..af0a7213e 100644
--- a/lib/puppet/parser/functions/sprintf.rb
+++ b/lib/puppet/parser/functions/sprintf.rb
@@ -5,9 +5,9 @@ module Puppet::Parser::Functions
 
         :doc => "Perform printf-style formatting of text.
 
-            The first parameter is format string describing how the rest of the parameters should be formatted.  See the documentation for the ``Kernel::sprintf()`` function in Ruby for all the details.") do |args|
+            The first parameter is format string describing how the rest of the parameters should be formatted.  See the documentation for the ``Kernel::sprintf`` function in Ruby for all the details.") do |args|
         raise Puppet::ParseError, 'sprintf() needs at least one argument' if args.length < 1
-        fmt = args.shift()
+        fmt = args.shift
         return sprintf(fmt, *args)
     end
 end