Optimize fcontext parsing methods

lib/puppet/provider/selinux_fcontext/semanage.rb

@@ -27,10 +27,9 @@ def self.file_type_map(val)
   end
 
   def self.parse_fcontext_lines(lines)
-    ret = []
-    lines.each do |line|
+    lines.filter_map do |line|
       next if line.strip.empty?
-      next if line =~ %r{^#}
+      next if line.start_with?('#')
 
       split = line.split(%r{\s+})
       if split.length == 2
@@ -45,16 +44,16 @@ def self.parse_fcontext_lines(lines)
         user = range = role = nil
       end
       ft = file_type_map(file_type)
-      ret.push(new(ensure: :present,
-                   name: "#{path_spec}_#{ft}",
-                   pathspec: path_spec,
-                   seltype: type,
-                   seluser: user,
-                   selrole: role,
-                   selrange: range,
-                   file_type: ft))
+
+      new(ensure: :present,
+          name: "#{path_spec}_#{ft}",
+          pathspec: path_spec,
+          seltype: type,
+          seluser: user,
+          selrole: role,
+          selrange: range,
+          file_type: ft)
     end
-    ret
   end
 
   def self.parse_fcontext_file(path)

lib/puppet/provider/selinux_fcontext_equivalence/semanage.rb

@@ -11,17 +11,16 @@
   mk_resource_methods
 
   def self.parse_fcontext_subs_lines(lines)
-    ret = []
-    lines.each do |line|
+    lines.filter_map do |line|
       next if line.strip.empty?
-      next if line =~ %r{^#}
+      next if line.start_with?('#')
 
       source, target = line.split(%r{\s+})
-      ret.push(new(ensure: :present,
-                   name: source,
-                   target: target))
+
+      new(ensure: :present,
+          name: source,
+          target: target)
     end
-    ret
   end
 
   def self.instances