+1, with my obligatory:
def self.instances(base = '/')
return self.new(:name => base, :recurse => true, :recurselimit => 1,
:audit => :all).recurse_local.values
end
could more concisely be written:
def self.instances(base = '/')
new(:name => base, :recurse => true, :recurselimit => 1, :audit =>
:all).recurse_local.values
end
...or I suppose, to show that I can see both sides of the issue, more
verbosely written:
def self.instances(base = '/')
options = {:name => base, :recurse => true, :recurselimit => 1, :audit
=> :all}
result = self.new(options).recurse_local.values
return result
end
-- M
-----------------------------------------------------------
The power of accurate observation is
commonly called cynicism by those
who have not got it. ~George Bernard Shaw
------------------------------------------------------------
--
You received this message because you are subscribed to the Google Groups
"Puppet Developers" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/puppet-dev?hl=en.
