maint: Make args to Catalog.select explicit

No other args are used, so passing *args and shifting is unnecessary.

2 files changed, 9 insertions, 9 deletions

diff --git a/README.markdown b/README.markdown
index dd9dd5867..656a3fbad 100644
--- a/README.markdown
+++ b/README.markdown
@@ -96,12 +96,14 @@ Like most parts of Puppet, these are easy to extend.  Just drop a new action int
 
     $ cat lib/puppet/interface/catalog/select.rb 
     # Select and show a list of resources of a given type.
-    Puppet::Interface::Catalog.action :select do |*args|
-      host = args.shift
-      type = args.shift
-      catalog = Puppet::Resource::Catalog.indirection.find(host)
-
-      catalog.resources.reject { |res| res.type != type }.each { |res| puts res }
+    Puppet::Interface.interface(:catalog) do
+      action :select do
+        invoke do |host,type|
+          catalog = Puppet::Resource::Catalog.indirection.find(host)
+
+          catalog.resources.reject { |res| res.type != type }.each { |res| puts res }
+        end
+      end
     end
     $ puppet catalog select localhost Class
     Class[main]
diff --git a/lib/puppet/interface/catalog/select.rb b/lib/puppet/interface/catalog/select.rb
index 3f724d360..32d9b7c45 100644
--- a/lib/puppet/interface/catalog/select.rb
+++ b/lib/puppet/interface/catalog/select.rb
@@ -1,9 +1,7 @@
 # Select and show a list of resources of a given type.
 Puppet::Interface.interface(:catalog) do
   action :select do
-    invoke do |*args|
-      host = args.shift
-      type = args.shift
+    invoke do |host,type|
       catalog = Puppet::Resource::Catalog.indirection.find(host)
 
       catalog.resources.reject { |res| res.type != type }.each { |res| puts res }