diff options
author | David Lutterkort <lutter@redhat.com> | 2009-05-30 23:46:11 -0700 |
---|---|---|
committer | James Turnbull <james@lovedthanlost.net> | 2009-06-03 23:34:53 +1000 |
commit | 386923ea95f161683fbdc2fe182193f3f3fe5485 (patch) | |
tree | 8c1ebacbb4456b0dbab25599e56c1ea1e1e79849 /lib | |
parent | 171669ad3d3d1f0b7f72cd5cb2c36ad97e7d315c (diff) | |
download | puppet-386923ea95f161683fbdc2fe182193f3f3fe5485.tar.gz puppet-386923ea95f161683fbdc2fe182193f3f3fe5485.tar.xz puppet-386923ea95f161683fbdc2fe182193f3f3fe5485.zip |
* provider/augeas: remove useless checks for nil
Diffstat (limited to 'lib')
-rw-r--r-- | lib/puppet/provider/augeas/augeas.rb | 68 |
1 files changed, 32 insertions, 36 deletions
diff --git a/lib/puppet/provider/augeas/augeas.rb b/lib/puppet/provider/augeas/augeas.rb index 77e261609..85dcd03e2 100644 --- a/lib/puppet/provider/augeas/augeas.rb +++ b/lib/puppet/provider/augeas/augeas.rb @@ -165,16 +165,14 @@ Puppet::Type.type(:augeas).provide(:augeas) do #check the value in augeas result = @aug.get(path) || '' - unless result.nil? - case comparator - when "!=" - return_value = (result != arg) - when "=~" - regex = Regexp.new(arg) - return_value = (result =~ regex) - else - return_value = (result.send(comparator, arg)) - end + case comparator + when "!=" + return_value = (result != arg) + when "=~" + regex = Regexp.new(arg) + return_value = (result =~ regex) + else + return_value = (result.send(comparator, arg)) end return !!return_value end @@ -196,32 +194,30 @@ Puppet::Type.type(:augeas).provide(:augeas) do #Get the values from augeas result = @aug.match(path) || '' # Now do the work - unless result.nil? - case verb - when "size": - fail("Invalid command: #{cmd_array.join(" ")}") if clause_array.length != 2 - comparator = clause_array.shift - arg = clause_array.shift - return_value = (result.size.send(comparator, arg)) - when "include": - arg = clause_array.shift - return_value = result.include?(arg) - when "eq": - begin - arg = clause_array.shift - new_array = eval arg - return_value = (result == new_array) - rescue - fail("Invalid array in command: #{cmd_array.join(" ")}") - end - when "noteq": - begin - arg = clause_array.shift - new_array = eval arg - return_value = (result != new_array) - rescue - fail("Invalid array in command: #{cmd_array.join(" ")}") - end + case verb + when "size" + fail("Invalid command: #{cmd_array.join(" ")}") if clause_array.length != 2 + comparator = clause_array.shift + arg = clause_array.shift + return_value = (result.size.send(comparator, arg)) + when "include" + arg = clause_array.shift + return_value = result.include?(arg) + when "eq" + begin + arg = clause_array.shift + new_array = eval arg + return_value = (result == new_array) + rescue + fail("Invalid array in command: #{cmd_array.join(" ")}") + end + when "noteq" + begin + arg = clause_array.shift + new_array = eval arg + return_value = (result != new_array) + rescue + fail("Invalid array in command: #{cmd_array.join(" ")}") end end return !!return_value |