diff options
author | Markus Roberts <Markus@reality.com> | 2010-07-09 18:06:58 -0700 |
---|---|---|
committer | Markus Roberts <Markus@reality.com> | 2010-07-09 18:06:58 -0700 |
commit | 8f15707251cdb58d53e82c4bbd332a38c2d31b4c (patch) | |
tree | b4a5df05fb8121c498cd33b67516d69af3d9853b /lib/puppet/indirector.rb | |
parent | c3e2353afb7fc2fb12efd1eb2bc5c342c792fb3b (diff) | |
download | puppet-8f15707251cdb58d53e82c4bbd332a38c2d31b4c.tar.gz puppet-8f15707251cdb58d53e82c4bbd332a38c2d31b4c.tar.xz puppet-8f15707251cdb58d53e82c4bbd332a38c2d31b4c.zip |
Code smell: Don't restate results directly after assignment
Replaced 33 occurances of
([$@]?\w+)( +[|&+-]{0,2}= .+)
\1
end
with
3 Examples:
The code:
@sync ||= Sync.new
@sync
end
becomes:
@sync ||= Sync.new
end
The code:
str += "\n"
str
end
becomes:
str += "\n"
end
The code:
@indirection = Puppet::Indirector::Indirection.new(self, indirection, options)
@indirection
end
becomes:
@indirection = Puppet::Indirector::Indirection.new(self, indirection, options)
end
Diffstat (limited to 'lib/puppet/indirector.rb')
-rw-r--r-- | lib/puppet/indirector.rb | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/puppet/indirector.rb b/lib/puppet/indirector.rb index 125445d23..1caad6c4b 100644 --- a/lib/puppet/indirector.rb +++ b/lib/puppet/indirector.rb @@ -28,7 +28,6 @@ module Puppet::Indirector # instantiate the actual Terminus for that type and this name (:ldap, w/ args :node) # & hook the instantiated Terminus into this class (Node: @indirection = terminus) @indirection = Puppet::Indirector::Indirection.new(self, indirection, options) - @indirection end module ClassMethods |