From 93ebe94656230209315ce0214103b239a28dec5e Mon Sep 17 00:00:00 2001
From: Jacob Helwig <jacob@technosorcery.net>
Date: Mon, 25 Jun 2018 11:07:12 -0700
Subject: Remove Rubocop Style/GuardClause violation

By restructuring this code to use a guard clause style, we can also get
rid of the assignment in conditional.
---
 lib/puppet/provider/ssh_authorized_key/parsed.rb | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

(limited to 'lib')

diff --git a/lib/puppet/provider/ssh_authorized_key/parsed.rb b/lib/puppet/provider/ssh_authorized_key/parsed.rb
index 45ef649..02a19eb 100644
--- a/lib/puppet/provider/ssh_authorized_key/parsed.rb
+++ b/lib/puppet/provider/ssh_authorized_key/parsed.rb
@@ -76,12 +76,13 @@ Puppet::Type.type(:ssh_authorized_key).provide(
     until scanner.eos?
       scanner.skip(%r{[ \t]*})
       # scan a long option
-      if (out = scanner.scan(%r{[-a-z0-9A-Z_]+=\".*?[^\\]\"})) || (out = scanner.scan(%r{[-a-z0-9A-Z_]+}))
-        result << out
-      else
-        # found an unscannable token, let's abort
-        break
-      end
+      out = scanner.scan(%r{[-a-z0-9A-Z_]+=\".*?[^\\]\"}) || scanner.scan(%r{[-a-z0-9A-Z_]+})
+
+      # found an unscannable token, let's abort
+      break unless out
+
+      result << out
+
       # eat a comma
       scanner.skip(%r{[ \t]*,[ \t]*})
     end
-- 
cgit v1.2.3