- networking_interfaces.each do |name,interface|
- if options[:role].nil? or interface[:role].to_s == options[:role].to_s
- if options[:family].nil? or interface[:family].to_s == options[:family].to_s
- if block.nil?
- interfaces << interface
- else
- block.call(interface)
- end
- end
+ networking_interfaces.each_value do |interface|
+ next unless options[:role].nil? || interface[:role].to_s == options[:role].to_s
+ next unless options[:family].nil? || interface[:family].to_s == options[:family].to_s
+
+ if block.nil?
+ interfaces << interface
+ else
+ yield interface