Layout/EmptyLinesAroundArguments

This commit enables the Rubocop Layout/EmptyLinesAroundArguments cop.

Author: mhashizume

.rubocop_todo.yml

@@ -30,19 +30,6 @@ Layout/EmptyLineAfterGuardClause:
 Layout/EmptyLineAfterMagicComment:
   Enabled: false
 
-# This cop supports safe auto-correction (--auto-correct).
-Layout/EmptyLinesAroundArguments:
-  Exclude:
-    - 'lib/puppet/defaults.rb'
-    - 'lib/puppet/parser/functions/sprintf.rb'
-    - 'lib/puppet/settings.rb'
-    - 'lib/puppet/type.rb'
-    - 'lib/puppet/type/schedule.rb'
-    - 'lib/puppet/util/diff.rb'
-    - 'lib/puppet/util/rdoc/generators/puppet_generator.rb'
-    - 'lib/puppet/util/rdoc/generators/template/puppet/puppet.rb'
-    - 'lib/puppet/util/windows/com.rb'
-
 # This cop supports safe auto-correction (--auto-correct).
 # Configuration parameters: AllowAliasSyntax, AllowedMethods.
 # AllowedMethods: alias_method, public, protected, private

lib/puppet/defaults.rb

@@ -813,7 +813,6 @@ def self.initialize_default_settings!(settings)
 
     settings.define_settings(
     :main,
-
     # We have to downcase the fqdn, because the current ssl stuff (as opposed to in master) doesn't have good facilities for
     # manipulating naming.
     :certname => {

lib/puppet/parser/functions/sprintf.rb

@@ -42,7 +42,6 @@
   ```
 
   This statement produces a notice of `value is : 42`."
-
 ) do |args|
   fmt = args[0]
   args = args[1..-1]

lib/puppet/settings.rb

@@ -1090,7 +1090,6 @@ def to_config
 The file format supports octothorpe-commented lines, but not partial-line comments.
 
 Generated on #{Time.now}.
-
 }.gsub(/^/, "# ")
 
     #         Add a section heading that matches our name.

lib/puppet/type.rb

@@ -2281,7 +2281,6 @@ def self.validate(&block)
   def log(msg)
 
     Puppet::Util::Log.create(
-
       :level => @parameters[:loglevel].value,
       :message => msg,
 

lib/puppet/type/schedule.rb

@@ -400,7 +400,6 @@ def self.mkdefaultschedules
       Puppet.debug "Creating default schedules"
 
             result << self.new(
-
         :name => "puppet",
         :period => :hourly,
 

lib/puppet/util/diff.rb

@@ -48,7 +48,6 @@ def lcs_diff(data_old, data_new, format=:unified, context_lines=3)
         hunk = ::Diff::LCS::Hunk.new(
           data_old, data_new, piece,
           context_lines,
-
           file_length_difference)
         file_length_difference = hunk.file_length_difference
       next unless oldhunk

lib/puppet/util/rdoc/generators/puppet_generator.rb

@@ -242,7 +242,6 @@ def gen_class_index
           gen_composite_index(
             file,
             RDoc::Page::COMBO_INDEX,
-
             "#{MODULE_DIR}/fr_#{file["file"].context.module_name}.html")
         end
       end
@@ -502,7 +501,6 @@ def write_on(f)
         template = TemplatePage.new(
           RDoc::Page::BODYINC,
           RDoc::Page::NODE_PAGE,
-
           RDoc::Page::METHOD_LIST)
       template.write_html_on(f, @values)
     end
@@ -746,7 +744,6 @@ def write_on(f)
         template = TemplatePage.new(
           RDoc::Page::BODYINC,
           RDoc::Page::PLUGIN_PAGE,
-
           RDoc::Page::PLUGIN_LIST)
       template.write_html_on(f, @values)
     end

lib/puppet/util/rdoc/generators/template/puppet/puppet.rb

@@ -849,11 +849,9 @@ module Page
 !INCLUDE!  <!-- banner header -->
 
   <div id="bodyContent">
-
 } +  METHOD_LIST + %{
 
   </div>
-
 } + FOOTER
 
 BODYINC = HEADER + %{
@@ -865,7 +863,6 @@ module Page
 !INCLUDE!
 
   </div>
-
 } + FOOTER
 
 #####################################################################

lib/puppet/util/windows/com.rb

@@ -196,7 +196,6 @@ def initialize(opts = {})
 
   IUnknown = Interface[
     FFI::WIN32::GUID['00000000-0000-0000-C000-000000000046'],
-
     QueryInterface: [[:pointer, :pointer], :hresult],
     AddRef: [[], :win32_ulong],
     Release: [[], :win32_ulong]