origen_testers 0.52.15 → 0.53.0

data/lib/origen_testers/igxl_based_tester/base/flow_line.rb

@@ -3,6 +3,7 @@ module OrigenTesters
     class Base
       class FlowLine
         attr_accessor :type, :id, :cz_setup # cz_setup is a virtual attrib since it is not part of the regular flow line
+
         # cz_setup combine with instance name when characterize opcode is used
 
         # Map any aliases to the official names here, multiple aliases for a given attribute
@@ -24,30 +25,23 @@ module OrigenTesters
           # Aliases can also be used to set defaults on multiple attributes like this,
           # use :value to refer to the value passed in to the given alias
           flag_false:     { device_condition: 'flag-false',
-                            device_name:      :value
-                         },
+                            device_name:      :value },
           flag_true:      { device_condition: 'flag-true',
-                            device_name:      :value
-                         },
+                            device_name:      :value },
           flag_false_any: { group_specifier: 'any-active',
                             group_condition: 'flag-false',
-                            group_name:      :value
-                             },
+                            group_name:      :value },
           flag_false_all: { group_specifier: 'all-active',
                             group_condition: 'flag-false',
-                            group_name:      :value
-                             },
+                            group_name:      :value },
           flag_true_any:  { group_specifier: 'any-active',
                             group_condition: 'flag-true',
-                            group_name:      :value
-                            },
+                            group_name:      :value },
           flag_true_all:  { group_specifier: 'all-active',
                             group_condition: 'flag-true',
-                            group_name:      :value
-                            },
+                            group_name:      :value },
           flag_clear:     { device_condition: 'flag-clear',
-                            device_name:      :value
-                         }
+                            device_name:      :value }
         }
 
         # Assign attribute defaults here, generally this should match whatever defaults
@@ -155,11 +149,11 @@ module OrigenTesters
           @type = type
           # Set the defaults
           DEFAULTS[@type.to_sym].each do |k, v|
-            send("#{k}=", v) if self.respond_to?("#{k}=")
+            send("#{k}=", v) if respond_to?("#{k}=")
           end
           # Then the values that have been supplied
           attrs.each do |k, v|
-            send("#{k}=", v) if self.respond_to?("#{k}=")
+            send("#{k}=", v) if respond_to?("#{k}=")
           end
           # override test numbers if diff-friendly output desired
           if Origen.tester.diff_friendly_output?
@@ -257,7 +251,7 @@ module OrigenTesters
           # code
         end
 
-        def id
+        def id # rubocop:disable Lint/DuplicateMethods
           @id || "#{parameter}_#{unique_counter}"
         end
 

data/lib/origen_testers/igxl_based_tester/base/generator.rb

@@ -337,6 +337,7 @@ module OrigenTesters
           [edgeset_sheets, timeset_sheets].each do |sheets|
             sheets.each do |name, sheet|
               next if sheet.class.name =~ /Edgesets$/ && sheet.ts_basic
+
               g << sheet
             end
           end
@@ -362,6 +363,7 @@ module OrigenTesters
         def pinmaps(filename = pinmaps_filename)
           f = filename.to_sym
           return pinmap_sheets[f] if pinmap_sheets[f]
+
           p = platform::Pinmap.new
           p.filename = f
           pinmap_sheets[f] = p
@@ -377,6 +379,7 @@ module OrigenTesters
         def test_instances(filename = test_instances_filename)
           f = filename.to_sym
           return test_instance_sheets[f] if test_instance_sheets[f]
+
           t = platform::TestInstances.new
           t.filename = f
           test_instance_sheets[f] = t
@@ -391,6 +394,7 @@ module OrigenTesters
         def patsets(filename = patsets_filename)
           f = filename.to_sym
           return patset_sheets[f] if patset_sheets[f]
+
           p = platform::Patsets.new
           p.filename = f
           patset_sheets[f] = p
@@ -407,6 +411,7 @@ module OrigenTesters
         def patsubrs(filename = patsubrs_filename)
           f = filename.to_sym
           return patsubr_sheets[f] if patsubr_sheets[f]
+
           p = platform::Patsubrs.new
           p.filename = f
           patsubr_sheets[f] = p
@@ -426,6 +431,7 @@ module OrigenTesters
             # DH here need to reset the flow!!
             f = filename.to_sym
             return flow_sheets[f] if flow_sheets[f] # will return flow if already existing
+
             p = platform::Flow.new
             p.inhibit_output if Origen.interface.resources_mode?
             p.filename = f
@@ -442,6 +448,7 @@ module OrigenTesters
         def patgroups(filename = patgroups_filename)
           f = filename.to_sym
           return patgroup_sheets[f] if patgroup_sheets[f]
+
           p = platform::Patgroups.new
           p.filename = f
           patgroup_sheets[f] = p
@@ -470,6 +477,7 @@ module OrigenTesters
 
           f = filename.to_sym
           return edgeset_sheets[f] if edgeset_sheets[f]
+
           e = platform::Edgesets.new(options)
           e.filename = f
           edgeset_sheets[f] = e
@@ -489,6 +497,7 @@ module OrigenTesters
 
           f = filename.to_sym
           return timeset_sheets[f] if timeset_sheets[f]
+
           case options[:timeset_basic]
           when true
             t = platform::TimesetsBasic.new(options)
@@ -518,12 +527,13 @@ module OrigenTesters
         def levelsets(filename = levelsets_filename)
           f = filename.to_sym
           return levelset_sheets[f] if levelset_sheets[f]
+
           t = platform::Levelset.new
           t.filename = f
           levelset_sheets[f] = t
         end
-        alias_method :time_sets, :timesets
-        alias_method :timing_sets, :timesets
+        alias_method :time_sets, :timesets # rubocop:disable Lint/DuplicateMethods
+        alias_method :timing_sets, :timesets # rubocop:disable Lint/DuplicateMethods
 
         # Returns the current AC specset sheet (as defined by the current value of
         # ac_specsets_filename).
@@ -534,6 +544,7 @@ module OrigenTesters
         def ac_specsets(filename = ac_specsets_filename)
           f = filename.to_sym
           return ac_specset_sheets[f] if ac_specset_sheets[f]
+
           s = platform::ACSpecsets.new
           s.filename = f
           ac_specset_sheets[f] = s
@@ -548,6 +559,7 @@ module OrigenTesters
         def dc_specsets(filename = dc_specsets_filename)
           f = filename.to_sym
           return dc_specset_sheets[f] if dc_specset_sheets[f]
+
           s = platform::DCSpecsets.new
           s.filename = f
           dc_specset_sheets[f] = s
@@ -562,6 +574,7 @@ module OrigenTesters
         def global_specs(filename = global_specs_filename)
           f = filename.to_sym
           return global_spec_sheets[f] if global_spec_sheets[f]
+
           s = platform::GlobalSpecs.new
           s.filename = f
           global_spec_sheets[f] = s
@@ -576,6 +589,7 @@ module OrigenTesters
         def program_jobs(filename = jobs_filename)
           f = filename.to_sym
           return job_sheets[f] if job_sheets[f]
+
           j = platform::Jobs.new
           j.filename = f
           job_sheets[f] = j
@@ -590,6 +604,7 @@ module OrigenTesters
         def references(filename = references_filename)
           f = filename.to_sym
           return reference_sheets[f] if reference_sheets[f]
+
           r = platform::References.new
           r.filename = f
           reference_sheets[f] = r

data/lib/origen_testers/igxl_based_tester/base/global_specs.rb

@@ -48,7 +48,7 @@ module OrigenTesters
           case data
           when NilClass
             data_new = 0
-          when Fixnum, Float
+          when Integer, Float
             case
             when data == 0
               data_new = data.to_s

data/lib/origen_testers/igxl_based_tester/base/job.rb

@@ -24,24 +24,24 @@ module OrigenTesters
 
         def initialize(jname, options = {}) # :nodoc:
           @name = jname
-          options[:pinmap] ? @pinmap         = [options[:pinmap]] : @pinmap         = []
-          options[:instances] ? @instances      = [options[:instances]] : @instances      = []
-          options[:flows] ? @flows          = [options[:flows]] : @flows          = []
-          options[:ac_specs] ? @ac_specs       = [options[:ac_specs]] : @ac_specs       = []
-          options[:dc_specs] ? @dc_specs       = [options[:dc_specs]] : @dc_specs       = []
-          options[:patsets] ? @patsets        = [options[:patsets]] : @patsets        = []
-          options[:patgroups] ? @patgroups      = [options[:patgroups]] : @patgroups      = []
-          options[:bintables] ? @bintables      = [options[:bintables]] : @bintables      = []
-          options[:cz] ? @cz             = [options[:cz]] : @cz             = []
-          options[:test_procs] ? @test_procs     = [options[:test_procs]] : @test_procs     = []
+          options[:pinmap] ? @pinmap = [options[:pinmap]] : @pinmap = []
+          options[:instances] ? @instances = [options[:instances]] : @instances = []
+          options[:flows] ? @flows = [options[:flows]] : @flows = []
+          options[:ac_specs] ? @ac_specs       = [options[:ac_specs]] : @ac_specs = []
+          options[:dc_specs] ? @dc_specs       = [options[:dc_specs]] : @dc_specs = []
+          options[:patsets] ? @patsets = [options[:patsets]] : @patsets = []
+          options[:patgroups] ? @patgroups      = [options[:patgroups]] : @patgroups = []
+          options[:bintables] ? @bintables      = [options[:bintables]] : @bintables = []
+          options[:cz] ? @cz = [options[:cz]] : @cz = []
+          options[:test_procs] ? @test_procs = [options[:test_procs]] : @test_procs = []
           options[:mix_sig_timing] ? @mix_sig_timing = [options[:mix_sig_timing]] : @mix_sig_timing = []
-          options[:wave_defs] ? @wave_defs      = [options[:wave_defs]] : @wave_defs      = []
-          options[:psets] ? @psets          = [options[:psets]] : @psets          = []
-          options[:signals] ? @signals        = [options[:signals]] : @signals        = []
-          options[:port_map] ? @port_map       = [options[:port_map]] : @port_map       = []
-          options[:fract_bus] ? @fract_bus      = [options[:fract_bus]] : @fract_bus      = []
+          options[:wave_defs] ? @wave_defs = [options[:wave_defs]] : @wave_defs = []
+          options[:psets] ? @psets = [options[:psets]] : @psets = []
+          options[:signals] ? @signals = [options[:signals]] : @signals = []
+          options[:port_map] ? @port_map = [options[:port_map]] : @port_map = []
+          options[:fract_bus] ? @fract_bus = [options[:fract_bus]] : @fract_bus = []
           options[:concurrent_seq] ? @concurrent_seq = [options[:concurrent_seq]] : @concurrent_seq = []
-          options[:comment] ? @comment        = options[:instances] : @comment        = nil
+          options[:comment] ? @comment = options[:instances] : @comment = nil
         end
 
         # Assigns job information for the given object

data/lib/origen_testers/igxl_based_tester/base/level_io_se.rb

@@ -17,8 +17,8 @@ module OrigenTesters
             vcl:       -1,            # Voltage clamp low
             vch:       2.5,            # Voltage clamp high
             vt:        0.9,            # Termination voltage
-            voutlotyp: 0,            #
-            vouthityp: 0,            #
+            voutlotyp: 0,
+            vouthityp: 0,
             dmode:     'Largeswing-VT' # Driver mode (possibly ATE-specific)
           }.merge(options)
           @vil       = options[:vil]

data/lib/origen_testers/igxl_based_tester/base/levelset.rb

@@ -18,8 +18,6 @@ module OrigenTesters
           @pins = {}
         end
 
-        # rubocop:disable Metrics/ParameterLists
-
         # Adds a pin level to the given levelset
         def add(lsname, pin, level, options = {})
           options = {
@@ -31,7 +29,7 @@ module OrigenTesters
           add_level(pin, level)
           @name = lsname
           @spec_sheet = options[:spec_sheet]
-          @ls_sheet_pins = options[:ls_sheet_pins] unless @ls_sheet_pins
+          @ls_sheet_pins ||= options[:ls_sheet_pins]
         end
 
         # Assigns a level object to the given pin for this levelset
@@ -90,8 +88,7 @@ module OrigenTesters
         end
 
         def format_uflex_level(data, options = {})
-          options = {
-          }.merge(options)
+          options = {}.merge(options)
 
           if data !~ /^\s*$/
             data = data.gsub(/^/, '=')

data/lib/origen_testers/igxl_based_tester/base/patgroup.rb

@@ -11,8 +11,7 @@ module OrigenTesters
             pattern: :pattern_file
           }
 
-          DEFAULTS = {
-          }
+          DEFAULTS = {}
 
           # Generate accessors for all attributes and their aliases
           ATTRS.each do |attr|
@@ -82,7 +81,6 @@ module OrigenTesters
         def name=(n)
           @name = n
           lines.each { |l| l.group_name = n }
-          n
         end
 
         # Returns all lines in the pattern set

data/lib/origen_testers/igxl_based_tester/base/patset.rb

@@ -37,7 +37,6 @@ module OrigenTesters
         def name=(n)
           @name = n
           lines.each { |l| l.pattern_set = n }
-          n
         end
 
         # Returns all lines in the pattern set

data/lib/origen_testers/igxl_based_tester/base/patsubr.rb

@@ -37,7 +37,6 @@ module OrigenTesters
         def name=(n)
           @name = n
           lines.each { |l| l.pattern_set = n }
-          n
         end
 
         # Returns all lines in the pattern set

data/lib/origen_testers/igxl_based_tester/base/test_instance.rb

@@ -52,19 +52,19 @@ module OrigenTesters
           # Build the type specific accessors (aliases)
           self.class::TEST_INSTANCE_ALIASES[@type.to_sym].each do |_alias, val|
             define_singleton_method("#{_alias}=") do |v|
-              send("#{val}=", v) if self.respond_to?("#{val}=", v)
+              send("#{val}=", v) if respond_to?("#{val}=", v)
             end
             define_singleton_method("#{_alias}") do
-              send(val) if self.respond_to?(val)
+              send(val) if respond_to?(val)
             end
           end
           # Set the defaults
           self.class::TEST_INSTANCE_DEFAULTS[@type.to_sym].each do |k, v|
-            send("#{k}=", v) if self.respond_to?("#{k}=", v)
+            send("#{k}=", v) if respond_to?("#{k}=", v)
           end
           # Then the values that have been supplied
           attrs.each do |k, v|
-            send("#{k}=", v) if self.respond_to?("#{k}=", v)
+            send("#{k}=", v) if respond_to?("#{k}=", v)
           end
         end
 
@@ -212,8 +212,7 @@ module OrigenTesters
               r = r / 1_000_000.0
             elsif r = options.delete(:ma) || options.delete(:mA)
               r = r / 1000.0
-            elsif r = options.delete(:a) || options.delete(:A)
-            else
+              r = options.delete(:a) || options.delete(:A)
               fail "Can't determine requested irange!"
             end
           end
@@ -229,7 +228,7 @@ module OrigenTesters
                 when r > 0.00002 then 2
                 when r > 0.000002 then 1
                 else 0
-                end
+                            end
             end
 
           elsif @type == :powersupply
@@ -246,21 +245,22 @@ module OrigenTesters
                 when r > 0.00005 then 2   # between 50ua - 500u
                 when r > 0.000005 then 3  # between 5u - 50u
                 else 8
-              end
+                            end
             end
 
           else # :pin_pmu
             if r == :smart
               self.irange = 5
             elsif r == :auto
-              fail 'Auto range not available in FIMV mode!' if self.fimv?
+              fail 'Auto range not available in FIMV mode!' if fimv?
+
               self.irange = 6
             else
               if fimv?
                 self.irange = case
                   when r > 0.0002 then 2
                   else 4
-                  end
+                              end
               else
                 self.irange = case
                   when r > 0.0002 then 2
@@ -268,7 +268,7 @@ module OrigenTesters
                   when r > 0.000002 then 0
                   when r > 0.0000002 then 1
                   else 3
-                  end
+                              end
               end
             end
           end
@@ -297,8 +297,7 @@ module OrigenTesters
           elsif r == :auto
             self.vrange = 5
           elsif !r
-            if r = options.delete(:v) || options.delete(:V)
-            else
+            unless r = options.delete(:v) || options.delete(:V)
               fail "Can't determine requested vrange!"
             end
           end
@@ -307,7 +306,7 @@ module OrigenTesters
             when r > 5 then 2
             when r > 2 then 1
             else 0
-            end
+                        end
           self
         end
 

data/lib/origen_testers/igxl_based_tester/base/test_instance_group.rb

@@ -12,7 +12,7 @@ module OrigenTesters
           @append_version = true
         end
 
-        def name
+        def name # rubocop:disable Lint/DuplicateMethods
           if unversioned_name
             if version && @append_version
               "#{unversioned_name}_v#{version}"
@@ -48,7 +48,7 @@ module OrigenTesters
           self.class == other_instance_group.class &&
             unversioned_name.to_s == other_instance_group.unversioned_name.to_s &&
             size == other_instance_group.size &&
-            self.all? do |ins|
+            all? do |ins|
               other_instance_group.any? { |other_ins| ins == other_ins }
             end
         end

data/lib/origen_testers/igxl_based_tester/base/timesets.rb

@@ -21,7 +21,7 @@ module OrigenTesters
           pin = pin.to_sym unless pin.is_a? Symbol
           esname = pin.to_sym unless esname.is_a? Symbol
           @ts.key?(tsname) ? @ts[tsname].add_edge(pin, esname) : @ts[tsname] = platform::Timeset.new(tsname, pin, esname, options)
-          @ts_sheet_pins = options[:ts_sheet_pins] unless @ts_sheet_pins
+          @ts_sheet_pins ||= options[:ts_sheet_pins]
           @ts[tsname]
         end
 

data/lib/origen_testers/igxl_based_tester/base/timesets_basic.rb

@@ -21,7 +21,7 @@ module OrigenTesters
           pin = pin.to_sym unless pin.is_a? Symbol
           esname = pin.to_sym unless esname.is_a? Symbol
           @ts.key?(tsname) ? @ts[tsname].add_edge(pin, esname) : @ts[tsname] = platform::Timeset.new(tsname, pin, esname, options)
-          @ts_sheet_pins = options[:ts_sheet_pins] unless @ts_sheet_pins
+          @ts_sheet_pins ||= options[:ts_sheet_pins]
           @ts[tsname]
         end
 

data/lib/origen_testers/igxl_based_tester/base.rb

@@ -167,9 +167,7 @@ module OrigenTesters
       def get_tester_channel(chanmapname, pinname, sitenum)
         if sitenum <= @max_site
           @testerchannel = @channelmap[chanmapname][sitenum][pinname].channel
-          return @testerchannel
-        else
-          return nil
+          @testerchannel
         end
       end
 
@@ -179,10 +177,8 @@ module OrigenTesters
         if sitenum <= @max_site
           if @channelmap[chanmapname][sitenum][pinname].type.include?('Merged')
             @merged_channels = @channelmap[chanmapname][sitenum][pinname].type.split('Merged')[1]
-            return 'x' + @merged_channels
+            'x' + @merged_channels
           end
-        else
-          return nil
         end
       end
 
@@ -206,12 +202,13 @@ module OrigenTesters
         if @testerconfig[testconfigname][slot][:instrument].to_s == 'VHDVS'
           @productnum = @testerconfig[testconfigname][slot][:idprom].split(' ')[0]
           # binding.pry
-          if (@productnum.include?('805-052-')) && (@productnum.split('-')[2].to_i >= 05)
+          if (@productnum.include?('805-052-')) && (@productnum.split('-')[2].to_i >= 0o5)
             return '+'
           end
         end
         nil  # if nothing matched
       end
+
       # Check if a specific VHDVS (UVS256) channel assignment is _HC variety (high-current)
       # If the specific VHDVS channel is _HC variety, returns a "_HC" string, otherwise nil.
       def is_vhdvs_hc(chanmapname, pinname, sitenum)
@@ -269,6 +266,7 @@ module OrigenTesters
         unless prog_dir
           fail 'You must supply the directory containing the test program sheets, or define it via Origen.config.test_program_output_directory'
         end
+
         @parser ||= IGXLBasedTester::Parser.new
         @parsed_dir ||= false
         if @parsed_dir != prog_dir
@@ -299,8 +297,7 @@ module OrigenTesters
       #   $tester.store(:offset => -2) # Just realized I need to capture that earlier vector
       def store(*pins)
         options = pins.last.is_a?(Hash) ? pins.pop : {}
-        options = { offset: 0
-                  }.merge(options)
+        options = { offset: 0 }.merge(options)
         update_vector microcode: 'stv', offset: options[:offset]
         last_vector(options[:offset]).contains_capture = true unless @inhibit_vectors
       end
@@ -319,8 +316,7 @@ module OrigenTesters
       #   $tester.cycle                # This is the vector that will be captured
       def store_next_cycle(*pins)
         options = pins.last.is_a?(Hash) ? pins.pop : {}
-        options = {
-        }.merge(options)
+        options = {}.merge(options)
 
         unless @inhibit_vectors
           preset_next_vector microcode: 'stv' do |vector|
@@ -407,8 +403,7 @@ module OrigenTesters
       #   $tester.freq_count($top.pin(:d_out))                 # Freq measure on pin "d_out"
       #   $tester.freq_count($top.pin(:d_out):readcode => 10)
       def freq_count(pin, options = {})
-        options = { readcode: false
-                  }.merge(options)
+        options = { readcode: false }.merge(options)
 
         set_code(options[:readcode]) if options[:readcode]
         cycle(microcode: "#{@microcode[:set_flag]} (cpuA)")
@@ -443,7 +438,7 @@ module OrigenTesters
           capture_vector_mem1: false,       # capture vector to memory type 1, here for J750 will be stv_m1
           capture_vector_mem2: false,       # capture vector to memory type 2, here for J750 will be stv_c
           pin:                 false,                       # pin on which to drive or expect data, pass pin object here!
-          pin_data:            false,                  # pin data (:none, :drive, :expect)
+          pin_data:            false                  # pin data (:none, :drive, :expect)
         }.merge(options)
 
         mto_opcode = ''
@@ -687,7 +682,7 @@ module OrigenTesters
           group:          false,            # If true the end pattern is intended to run within a pattern group
           high_voltage:   false,         # Supply a pin name here to declare it as an HV instrument (not yet defined)
           freq_counter:   false,     # Supply a pin name here to declare it as a frequency counter
-          memory_test:    false,      # If true, define 2-bit MTO DGEN as instrument
+          memory_test:    false      # If true, define 2-bit MTO DGEN as instrument
         }.merge(options)
 
         if level_period?
@@ -850,8 +845,9 @@ module OrigenTesters
       end
 
       def enable_flag(options = {})
-        options = { flagnum: 4,      # default flag to use
-                }.merge(options)
+        options = {
+          flagnum: 4 # default flag to use
+        }.merge(options)
 
         if options[:flagnum] > @flags.length
           abort "ERROR! Invalid flag value passed to 'enable_flag' method!\n"
@@ -861,8 +857,9 @@ module OrigenTesters
       end
 
       def set_flag(options = {})
-        options = { flagnum: 4,      # default flag to use
-                }.merge(options)
+        options = {
+          flagnum: 4 # default flag to use
+        }.merge(options)
 
         if options[:flagnum] > @flags.length
           abort "ERROR! Invalid flag value passed to 'set_flag' method!\n"

data/lib/origen_testers/igxl_based_tester/decompiler/atp.rb

@@ -43,42 +43,36 @@ module OrigenTesters
                                            pattern_header:       header,
                                            variable_assignments: variable_assignments,
                                            imports:              imports,
-                                           comments:             comments
-                                          )
+                                           comments:             comments)
         end
 
         def parse_pinlist(raw_pinlist:, context:)
           raw_pinlist = raw_pinlist.join('')
           OrigenTesters::Decompiler::Nodes::Pinlist.new(context: self,
-                                                        pins:    raw_pinlist[raw_pinlist.index('$') + 1..raw_pinlist.index(')') - 1].split(/,\s*/)[1..-1]
-                                                       )
+                                                        pins:    raw_pinlist[raw_pinlist.index('$') + 1..raw_pinlist.index(')') - 1].split(/,\s*/)[1..-1])
         end
 
         def parse_vector(raw_vector:, context:, meta:)
           if raw_vector =~ Regexp.new('^\s*//')
             nodes_namespace::CommentBlock.new(context:  self,
-                                              comments: raw_vector.split("\n")
-                                             )
+                                              comments: raw_vector.split("\n"))
           elsif raw_vector.strip.size == 0
             nodes_namespace::CommentBlock.new(context: self, comments: ['// blank line replaced with comment by origen convert'])
           elsif raw_vector =~ Regexp.new('^\s*start_label')
             nodes_namespace::StartLabel.new(context:     self,
-                                            start_label: raw_vector[raw_vector.index('start_label') + 11..-1].strip[0..-2]
-                                           )
+                                            start_label: raw_vector[raw_vector.index('start_label') + 11..-1].strip[0..-2])
           elsif raw_vector =~ Regexp.new('^\s*global')
             contents = raw_vector.strip['global'.size + 1..-2].strip.split(/\s+/)
             nodes_namespace::GlobalLabel.new(context:    self,
                                              label_type: contents[0],
-                                             label_name: contents[1]
-                                            )
+                                             label_name: contents[1])
           # original elsif for label was updated to avoid confusing origen's eol comments for a label
           # elsif raw_vector =~ Regexp.new(':(?!(.*>))')
           elsif raw_vector.split(';').first =~ Regexp.new(':(?!(.*>))')
             nodes_namespace::Label.new(context:    self,
                                        # Strip any whitespace from the vector and grab contents up to
                                        # the ':' symbol.
-                                       label_name: raw_vector.strip[0..-2]
-                                      )
+                                       label_name: raw_vector.strip[0..-2])
           else
 
             opcode_plus_args = raw_vector[0..(raw_vector.index('>') - 1)].rstrip.split(/\s+/)
@@ -88,14 +82,12 @@ module OrigenTesters
                                         pin_states:       timeset_plus_pins[1..-1],
                                         opcode:           (opcode_plus_args[0] && opcode_plus_args[0].empty?) ? nil : opcode_plus_args[0],
                                         opcode_arguments: opcode_plus_args[1..-1],
-                                        comment:          begin
-                if raw_vector =~ Regexp.new('//')
-                  raw_vector[raw_vector.index('//') + 2..-1].strip
-                else
-                  ''
-                end
-              end
-                                       )
+                                        comment:
+                                                          if raw_vector =~ Regexp.new('//')
+                                                            raw_vector[raw_vector.index('//') + 2..-1].strip
+                                                          else
+                                                            ''
+                                                          end)
           end
         end
       end

data/lib/origen_testers/igxl_based_tester/j750/flow_line.rb

@@ -8,8 +8,7 @@ module OrigenTesters
                                    sort_pass sort_fail result flag_pass flag_fail state
                                    group_specifier group_sense group_condition group_name
                                    device_sense device_condition device_name
-                                   debug_assume debug_sites comment
-                                )
+                                   debug_assume debug_sites comment)
 
         # Generate the instance method definitions based on the above
         define

data/lib/origen_testers/igxl_based_tester/j750/patset_pattern.rb

@@ -7,8 +7,7 @@ module OrigenTesters
         PATSET_ATTRS = %w(pattern_set file_name start_label stop_label comment)
 
         # Pattern set defaults
-        PATSET_DEFAULTS = {
-        }
+        PATSET_DEFAULTS = {}
 
         # Generate the instance method definitions based on the above
         define