srl_ruby 0.4.11 → 0.4.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3a2509164bd980bf2145d41c15281608694674f633b2a6f2b482b6298db7d36a
-  data.tar.gz: 3112162a2a27268dd4d0c1e8cf4b0af25b98a158152e7ceca9e9d5f34c347c96
+  metadata.gz: d1bd1a0bcdf03aad4c52a4df83bc252cf3f99ec3d9156c1d49a073f020f4c59c
+  data.tar.gz: 38de02e4dc7a6d7cf99a0d62f76adb89642e7785871834303f2c5df754d50dc4
 SHA512:
-  metadata.gz: 51099a1c625930c9e139079f73647524caf6db7ed23ccdcdee475a720dcdb238d4e48ad0632f6ec7b0cee8e57785a25ce7b5f4232c605811f382a95d11b31afa
-  data.tar.gz: a4a7383ce66944d5b5c834200a8e5b2367660912a93daaca13dc27eb95fd56f0b655317c575a4013dd5e69b22af804a4dcbfb5ce5a5804249bf78d7495d5a4e9
+  metadata.gz: 636462883d5d49e79bde56104fface3d68694a9e6e99ffd7329bb3f990707a4cf15373546ad93c7e0fb135605f50bce96b2cefe64936fbb690a57273f8103b7f
+  data.tar.gz: 561f4d2aed5fb6c425095ca666128fe00f15b72ddf84d6b27f5ab6778d0a6b0a5657e69b7c71651c11b7d346cbfc914b3c42bc491ae3626e04d9841828190ac4

data/.rubocop.yml

@@ -5,9 +5,9 @@ AllCops:
 
 Gemspec/DateAssignment:
   Enabled: true
-  
+
 Gemspec/RequireMFA: # new in 1.23
-  Enabled: false  
+  Enabled: false
 
 Layout/ArgumentAlignment:
   Enabled: false
@@ -134,9 +134,9 @@ Lint/RaiseException:
 
 Lint/RedundantDirGlobSort:
   Enabled: true
-  
+
 Lint/RefinementImportMethods: # new in 1.27
-  Enabled: true  
+  Enabled: true
 
 Lint/RequireRelativeSelfPath: # new in 1.22
   Enabled: true
@@ -167,9 +167,9 @@ Lint/UnusedMethodArgument:
 
 Lint/UselessAccessModifier:
   Enabled: true
-  
+
 Lint/UselessRuby2Keywords: # new in 1.23
-  Enabled: true  
+  Enabled: true
 
 Lint/Void:
   Enabled: false
@@ -215,9 +215,9 @@ Naming/ConstantName:
 
 Naming/ClassAndModuleCamelCase:
   Enabled: false
-  
+
 Naming/BlockForwarding: # new in 1.24
-  Enabled: true  
+  Enabled: true
 
 Naming/BlockParameterName:
   Enabled: true
@@ -234,6 +234,9 @@ Naming/MethodName:
 Naming/VariableName:
   Enabled: false
 
+Security/CompoundHash: # new in 1.28
+  Enabled: true
+
 Security/IoMethods: # new in 1.22
   Enabled: true
 
@@ -290,9 +293,12 @@ Style/ExpandPathArguments:
 
 Style/ExponentialNotation:
   Enabled: true
-  
+
+Style/FetchEnvVar: # new in 1.28
+  Enabled: true
+
 Style/FileRead: # new in 1.24
-  Enabled: true  
+  Enabled: true
 
 Style/FileWrite: # new in 1.24
   Enabled: true
@@ -320,9 +326,9 @@ Style/InPatternThen:
 
 Style/InverseMethods:
   Enabled: false
-  
+
 Style/MapToHash: # new in 1.24
-  Enabled: true  
+  Enabled: true
 
 Style/MissingRespondToMissing:
   Enabled: false
@@ -350,9 +356,12 @@ Style/NumberedParametersLimit: # new in 1.22
 
 Style/NumericLiterals:
   Enabled: false
-  
+
+Style/ObjectThen: # new in 1.28
+  Enabled: true
+
 Style/OpenStructUse: # new in 1.23
-  Enabled: true  
+  Enabled: true
 
 Style/QuotedSymbols:
   Enabled: true
@@ -362,12 +371,12 @@ Style/RaiseArgs:
 
 Style/RedundantArgument:
   Enabled: true
-  
+
 Style/RedundantInitialize: # new in 1.27
-  Enabled: true  
+  Enabled: true
 
 Style/RedundantReturn:
-  Enabled: false
+  Enabled: true
 
 Style/RedundantSelf:
   Enabled: true

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [0.4.12] - 2022-04-22
+- Code refactoring.
+
+### Changed
+- Refactoring class `SrlRuby::Tokenizer`: use of manifest constants, simplified newline and whitespace processing.
+- Updated of `.rubocop.yml` to integrate newer cops (from version 1.28)
+- Code refactoring: removal of redundant `return`
+
 ## [0.4.11] - 2022-04-17
 - Fixed code breaking change in Ruby 3.1+: prime library is no longer part of stdlib.
 

data/Rakefile

@@ -2,19 +2,18 @@
 
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task' # Rspec as testing tool
-# require 'cucumber/rake/task' # Cucumber as testing tool
+require 'cucumber/rake/task' # Cucumber as testing tool
 
 # desc 'Run RSpec'
 RSpec::Core::RakeTask.new(:spec)
 
-# Cucumber::Rake::Task.new do |_|
-  # # Comment
-# end
+Cucumber::Rake::Task.new do |_|
+  # Comment
+end
 
 # Combine RSpec and Cucumber tests
 desc 'Run tests, with RSpec and Cucumber'
-task test: :spec
-# task test: %i[spec cucumber]
+task test: %i[spec cucumber]
 
 
 # Default rake task

data/cucumber.yml

@@ -1,4 +1,4 @@
-default: --strict -v -b  --format progress features
+default: --strict -v -b --publish-quiet --format progress features
 
 
 

data/lib/regex/atomic_expression.rb

@@ -11,7 +11,7 @@ module Regex # This module is used as a namespace
     # Redefined method.
     # @return [TrueClass] Return true since it may not have any child.
     def atomic?
-      return true
+      true
     end
 
     # Notification that the parse tree construction is complete.

data/lib/regex/char_class.rb

@@ -30,9 +30,7 @@ module Regex # This module is used as a namespace
         end
         sub_result << child.to_str
       end
-      result = "[#{negated ? '^' : ''}#{result_children}]"
-
-      return result
+      "[#{negated ? '^' : ''}#{result_children}]"
     end
   end # class
 end # module

data/lib/regex/char_range.rb

@@ -44,7 +44,7 @@ module Regex # This module is used as a namespace
       msg = 'Character range error: lower bound is greater than upper bound.'
       raise StandardError, msg if theLowerBound.codepoint > theUpperBound.codepoint
 
-      return [theLowerBound, theUpperBound]
+      [theLowerBound, theUpperBound]
     end
   end # class
 end # module

data/lib/regex/char_shorthand.rb

@@ -45,7 +45,7 @@ module Regex # This module is used as a namespace
       msg = "Unknown predefined character class \\#{aShortname}"
       raise StandardError, msg unless StandardCClasses.include? aShortname
 
-      return aShortname
+      aShortname
     end
   end # class
 end # module

data/lib/regex/character.rb

@@ -107,9 +107,7 @@ module Regex # This module is used as a namespace
       msg = "Escape sequence #{esc_seq} does not begin with a backslash (\)."
       raise StandardError, msg unless esc_seq[0] == '\\'
 
-      result = (esc_seq.length == 2) ? digram2codepoint(esc_seq) : esc_number2codepoint(esc_seq)
-
-      return result
+      (esc_seq.length == 2) ? digram2codepoint(esc_seq) : esc_number2codepoint(esc_seq)
     end
 
     # Return the character as a String object
@@ -127,7 +125,7 @@ module Regex # This module is used as a namespace
     # newOne == 0x03a3  # true. The Integer is compared to the codepoint value.
     # Will test equality with any Object that knows the to_s method
     def ==(other)
-      result = case other
+      case other
         when Character
           to_str == other.to_str
 
@@ -141,8 +139,6 @@ module Regex # This module is used as a namespace
           # Unknown type: try with a convertion
           self == other.to_s # Recursive call
       end
-
-      return result
     end
 
     # Return a plain English description of the character
@@ -160,7 +156,7 @@ module Regex # This module is used as a namespace
     def text_repr
       return char if lexeme.nil?
 
-      return lexeme.dup
+      lexeme.dup
     end
 
     # Convertion method that returns a codepoint for the given two characters
@@ -178,7 +174,7 @@ module Regex # This module is used as a namespace
       # If it not a special sequence, then escaped character is
       # considered literally (the backslash is 'dummy')
       result = char2codepoint(aDigram[-1]) if result.nil?
-      return result
+      result
     end
 
     private_class_method :digram2codepoint
@@ -199,7 +195,7 @@ module Regex # This module is used as a namespace
 
         # Extract the hexadecimal number
         hexliteral = hexa # shorterSeq.sub(/^[xXu]\{?([0-9a-fA-F]+)}?$/, '\1')
-        return hexliteral.hex
+        hexliteral.hex
       end
     end
 

data/lib/regex/concatenation.rb

@@ -19,11 +19,9 @@ module Regex # This module is used as a namespace
     # Conversion method re-definition.
     # Purpose: Return the String representation of the concatented expressions.
     def text_repr
-      outcome = children.inject(+'') do |result, child|
+      children.inject(+'') do |result, child|
         result << child.to_str
       end
-
-      return outcome
     end
   end # class
 end # module

data/lib/regex/expression.rb

@@ -26,8 +26,7 @@ module Regex # This module is used as a namespace
     # @param theParentOptions [Hash] matching options. They are overridden
     # by options with same name that are bound to this object.
     def options(theParentOptions)
-      resulting_options = theParentOptions.merge(@local_options)
-      return resulting_options
+      theParentOptions.merge(@local_options)
     end
 
     # Template method.
@@ -38,7 +37,7 @@ module Regex # This module is used as a namespace
       result << text_repr
       result << suffix
 
-      return result
+      result
     end
 
     protected

data/lib/regex/lookaround.rb

@@ -42,8 +42,7 @@ module Regex # This module is used as a namespace
     def to_str
       dir_syntax = (dir == :ahead) ? '' : '<'
       kind_syntax = (kind == :positive) ? '=' : '!'
-      result = "(?#{dir_syntax}#{kind_syntax}#{child.to_str})"
-      return result
+      "(?#{dir_syntax}#{kind_syntax}#{child.to_str})"
     end
   end # class
 end # module

data/lib/regex/match_option.rb

@@ -26,7 +26,7 @@ module Regex # This module is used as a namespace
       result = 0
       flags.each { |f| result |= f }
 
-      return result
+      result
     end
 
     # Equality operator
@@ -40,7 +40,7 @@ module Regex # This module is used as a namespace
         isEqual = false
       end
 
-      return isEqual
+      isEqual
     end
 
     protected

data/lib/regex/monadic_expression.rb

@@ -32,9 +32,7 @@ module Regex # This module is used as a namespace
 
     # Return the text representation of the child (if any)
     def all_child_text
-      result = child.nil? ? '' : child.to_str
-
-      return result
+      child.nil? ? '' : child.to_str
     end
   end # class
 end # module

data/lib/regex/multiplicity.rb

@@ -56,7 +56,7 @@ module Regex # This module is used as a namespace
         possessive: '+'
       }
 
-      return subresult + policy2suffix[policy]
+      subresult + policy2suffix[policy]
     end
 
     private
@@ -66,7 +66,7 @@ module Regex # This module is used as a namespace
       err_msg = "Invalid lower bound of repetition count #{aLowerBound}"
       raise StandardError, err_msg unless aLowerBound.kind_of?(Integer)
 
-      return aLowerBound
+      aLowerBound
     end
 
     # Validation method. Return the validated lower bound value
@@ -76,7 +76,7 @@ module Regex # This module is used as a namespace
         raise StandardError, err_msg
       end
 
-      return anUpperBound
+      anUpperBound
     end
 
     # Validation method. Return the validated policy value.
@@ -85,7 +85,7 @@ module Regex # This module is used as a namespace
       valid_policies = %i[greedy lazy possessive]
       raise StandardError, err_msg unless valid_policies.include? aPolicy
 
-      return aPolicy
+      aPolicy
     end
   end # class
 end # module

data/lib/regex/non_capturing_group.rb

@@ -22,8 +22,7 @@ module Regex # This module is used as a namespace
     # Conversion method re-definition.
     # Purpose: Return the String representation of the captured expression.
     def text_repr
-      result = "(?:#{all_child_text})"
-      return result
+      "(?:#{all_child_text})"
     end
   end # class
 end # module

data/lib/regex/polyadic_expression.rb

@@ -24,7 +24,7 @@ module Regex # This module is used as a namespace
     def <<(aChild)
       @children << aChild
 
-      return self
+      self
     end
 
     # Notification that the parse tree construction is complete.
@@ -52,7 +52,7 @@ module Regex # This module is used as a namespace
     def df_visitor
       root = children # The visit will start from the children of this object
 
-      visitor = Enumerator.new do |result| # result is a Yielder
+      Enumerator.new do |result| # result is a Yielder
         # Initialization part: will run once
         visit_stack = [root] # The LIFO queue of nodes to visit
 
@@ -78,8 +78,6 @@ module Regex # This module is used as a namespace
           break if visit_stack.empty?
         end
       end
-
-      return visitor
     end
   end # class
 end # module

data/lib/regex/quantifiable.rb

@@ -8,7 +8,7 @@ module Regex # This module is used as a namespace
   module Quantifiable
     # Redefined method. Return true since it may not have any child.
     def quantified?
-      return !@quantifier.nil?
+      !@quantifier.nil?
     end
 
     def quantifier

data/lib/regex/repetition.rb

@@ -30,8 +30,7 @@ module Regex # This module is used as a namespace
     # Conversion method re-definition.
     # @return [String] String representation of the concatented expressions.
     def text_repr
-      result = all_child_text + multiplicity.to_str
-      return result
+      all_child_text + multiplicity.to_str
     end
   end # class
 end # module

data/lib/srl_ruby/ast_builder.rb

@@ -59,13 +59,11 @@ module SrlRuby
     # @param aTokenPosition [Integer] Position of token in the input stream
     # @param aToken [Rley::Lexical::Token] The input token
     def new_leaf_node(_production, _terminal, aTokenPosition, aToken)
-      node = Rley::PTree::TerminalNode.new(aToken, aTokenPosition)
-
-      return node
+      Rley::PTree::TerminalNode.new(aToken, aTokenPosition)
     end
 
     def multiplicity(lowerBound, upperBound)
-      return Regex::Multiplicity.new(lowerBound, upperBound, :greedy)
+      Regex::Multiplicity.new(lowerBound, upperBound, :greedy)
     end
 
     # rubocop: disable Style/OptionalBooleanParameter
@@ -88,14 +86,14 @@ module SrlRuby
         result = Regex::Character.new(aString)
       end
 
-      return result
+      result
     end
     # rubocop: enable Style/OptionalBooleanParameter
 
     def char_range(lowerBound, upperBound)
       lower = Regex::Character.new(lowerBound)
       upper = Regex::Character.new(upperBound)
-      return Regex::CharRange.new(lower, upper)
+      Regex::CharRange.new(lower, upper)
     end
 
     def char_class(toNegate, *theChildren)
@@ -283,7 +281,7 @@ module SrlRuby
       end
 
       # TODO check other implementations
-      return Regex::CharClass.new(false, *alternatives)
+      Regex::CharClass.new(false, *alternatives)
     end
 
     # rule('character_class' => %w[NONE OF STRING_LIT]).tag 'none_of'
@@ -337,13 +335,13 @@ module SrlRuby
       # What if literal is empty?...
 
       raw_literal = theChildren[-1].token.lexeme.dup
-      return string_literal(raw_literal)
+      string_literal(raw_literal)
     end
 
     # rule('raw' => %w[RAW STRING_LIT]).tag 'raw_literal'
     def reduce_raw_literal(_production, _range, _tokens, theChildren)
       raw_literal = theChildren[-1].token.lexeme.dup
-      return Regex::RawExpression.new(raw_literal)
+      Regex::RawExpression.new(raw_literal)
     end
 
     # rule('alternation' => %w[ANY OF LPAREN alternatives RPAREN]).tag 'any_of'
@@ -358,12 +356,12 @@ module SrlRuby
         result = Regex::Alternation.new(*theChildren[3])
       end
 
-      return result
+      result
     end
 
     # rule('alternatives' => %w[alternatives separator quantifiable]).tag 'alternative_list'
     def reduce_alternative_list(_production, _range, _tokens, theChildren)
-      return theChildren[0] << theChildren[-1]
+      theChildren[0] << theChildren[-1]
     end
 
     # rule('alternatives' => 'quantifiable').tag 'simple_alternative'
@@ -418,7 +416,7 @@ module SrlRuby
       make_last_repetition_lazy(theChildren[1])
       group = Regex::CapturingGroup.new(theChildren[1], name)
       (_, until_expr) = theChildren[4]
-      return Regex::Concatenation.new(group, until_expr)
+      Regex::Concatenation.new(group, until_expr)
     end
 
     # rule('quantifier' => 'ONCE').tag 'once'

data/lib/srl_ruby/tokenizer.rb

@@ -17,6 +17,16 @@ module SrlRuby
   # Delimiters: parentheses '(' and ')'
   # Separators: comma (optional)
   class Tokenizer
+    PATT_CHAR_CLASS = /[^,"\s]{2,}/.freeze
+    PATT_DIGIT_LIT = /[0-9]((?=\s|,|\))|$)/.freeze
+    PATT_IDENTIFIER = /[a-zA-Z_][a-zA-Z0-9_]+/.freeze
+    PATT_INTEGER = /[0-9]{2,}((?=\s|,|\))|$)/.freeze # An integer has 2..* digits
+    PATT_LETTER_LIT = /[a-zA-Z]((?=\s|,|\))|$)/.freeze
+    PATT_NEWLINE = /(?:\r\n)|\r|\n/.freeze
+    PATT_STR_DBL_QUOTE = /"(?:\\"|[^"])*"/.freeze # Double quotes literal?
+    PATT_STR_SNGL_QUOTE = /'(?:\\'|[^'])*'/.freeze # Single quotes literal?
+    PATT_WHITESPACE = /[ \t\f]+/.freeze
+
     # @return [StringScanner]
     attr_reader(:scanner)
 
@@ -26,14 +36,14 @@ module SrlRuby
     # @return [Integer] offset of start of current line within input
     attr_reader(:line_start)
 
-    @@lexeme2name = {
+    Lexeme2name = {
       '(' => 'LPAREN',
       ')' => 'RPAREN',
       ',' => 'COMMA'
     }.freeze
 
     # Here are all the SRL keywords (in uppercase)
-    @@keywords = %w[
+    Keywords = %w[
       ALL
       ALREADY
       AND
@@ -109,47 +119,55 @@ module SrlRuby
         tok_sequence << token unless token.nil?
       end
 
-      return tok_sequence
+      tok_sequence
     end
 
     private
 
     def _next_token
-      skip_whitespaces
-      curr_ch = scanner.peek(1)
-      return nil if curr_ch.nil? || curr_ch.empty?
-
       token = nil
 
-      if '(),'.include? curr_ch
-        # Delimiters, separators => single character token
-        token = build_token(@@lexeme2name[curr_ch], scanner.getch)
-      elsif (lexeme = scanner.scan(/[0-9]{2,}((?=\s|,|\))|$)/))
-        token = build_token('INTEGER', lexeme) # An integer has 2..* digits
-      elsif (lexeme = scanner.scan(/[0-9]((?=\s|,|\))|$)/))
-      token = build_token('DIGIT_LIT', lexeme)
-      elsif (lexeme = scanner.scan(/"(?:\\"|[^"])*"/)) # Double quotes literal?
-        unquoted = lexeme.gsub(/(^")|("$)/, '')
-        token = build_token('STRING_LIT', unquoted)
-      elsif (lexeme = scanner.scan(/'(?:\\'|[^'])*'/)) # Single quotes literal?
-        unquoted = lexeme.gsub(/(^')|('$)/, '')
-        token = build_token('STRING_LIT', unquoted)
-      elsif (lexeme = scanner.scan(/[a-zA-Z]((?=\s|,|\))|$)/))
-        token = build_token('LETTER_LIT', lexeme)
-      elsif (lexeme = scanner.scan(/[a-zA-Z_][a-zA-Z0-9_]+/))
-        keyw = @@keywords[lexeme.upcase]
-        tok_type = keyw || 'IDENTIFIER'
-        token = build_token(tok_type, lexeme)
-      elsif (lexeme = scanner.scan(/[^,"\s]{2,}/))
-        token = build_token('CHAR_CLASS', lexeme)
-      else # Unknown token
-        erroneous = curr_ch.nil? ? '' : scanner.scan(/./)
-        sequel = scanner.scan(/.{1,20}/)
-        erroneous += sequel unless sequel.nil?
-        raise ScanError, "Unknown token #{erroneous} on line #{lineno}"
-      end
+      # Loop until end of input reached or token found
+      until token || scanner.eos?
+
+        if scanner.skip(PATT_NEWLINE)
+          next_line_scanned
+          next
+        end
+        next if scanner.skip(PATT_WHITESPACE) # Skip whitespaces
+
+        curr_ch = scanner.peek(1)
+
+        token = if '(),'.include? curr_ch
+          # Delimiters, separators => single character token
+          build_token(Lexeme2name[curr_ch], scanner.getch)
+        elsif (lexeme = scanner.scan(PATT_INTEGER))
+          build_token('INTEGER', lexeme)
+        elsif (lexeme = scanner.scan(PATT_DIGIT_LIT))
+          build_token('DIGIT_LIT', lexeme)
+        elsif (lexeme = scanner.scan(PATT_STR_DBL_QUOTE))
+          unquoted = lexeme.gsub(/(^")|("$)/, '')
+          build_token('STRING_LIT', unquoted)
+        elsif (lexeme = scanner.scan(PATT_STR_SNGL_QUOTE))
+          unquoted = lexeme.gsub(/(^')|('$)/, '')
+          build_token('STRING_LIT', unquoted)
+        elsif (lexeme = scanner.scan(PATT_LETTER_LIT))
+          build_token('LETTER_LIT', lexeme)
+        elsif (lexeme = scanner.scan(PATT_IDENTIFIER))
+          keyw = Keywords[lexeme.upcase]
+          tok_type = keyw || 'IDENTIFIER'
+          build_token(tok_type, lexeme)
+        elsif (lexeme = scanner.scan(PATT_CHAR_CLASS))
+          build_token('CHAR_CLASS', lexeme)
+        else # Unknown token
+          erroneous = curr_ch.nil? ? '' : scanner.scan(/./)
+          sequel = scanner.scan(/.{1,20}/)
+          erroneous += sequel unless sequel.nil?
+          raise ScanError, "Unknown token #{erroneous} on line #{lineno}"
+        end
+      end # until
 
-      return token
+      token
     end
 
     def build_token(aSymbolName, aLexeme)
@@ -162,38 +180,13 @@ module SrlRuby
         raise e
       end
 
-      return token
-    end
-
-    def skip_whitespaces
-      pre_pos = scanner.pos
-
-      loop do
-        ws_found = false
-        found = scanner.skip(/[ \t\f]+/)
-        ws_found = true if found
-        found = scanner.skip(/(?:\r\n)|\r|\n/)
-        if found
-          ws_found = true
-          @lineno += 1
-          @line_start = scanner.pos
-        end
-        break unless ws_found
-      end
-
-      curr_pos = scanner.pos
-      return if curr_pos == pre_pos
-      # skipped = scanner.string.slice(Range.new(pre_pos, curr_pos))
-      # triplet = skipped.rpartition(/\n|\r/)
-      # @column = 1 unless triplet[1].empty?
-
-      # Correction for the tabs
-      # tab_count = triplet[2].chars.count { |ch| ch =~ /\t/ }
-      # @column += triplet[2].size + tab_count * (tab_size - 1) - 1
+      token
     end
 
-    def tab_size
-      2
+    # Event: next line detected.
+    def next_line_scanned
+      @lineno += 1
+      @line_start = scanner.pos
     end
   end # class
 end # module

data/lib/srl_ruby/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SrlRuby
-  VERSION = '0.4.11'
+  VERSION = '0.4.12'
 end

data/lib/srl_ruby.rb

@@ -14,7 +14,7 @@ module SrlRuby
     File.open(filename, 'r') { |f| source = f.read }
     return source if source.nil? || source.empty?
 
-    return parse(source)
+    parse(source)
   end
 
   # Compile the given SRL expression into its Regexp equivalent.
@@ -52,6 +52,6 @@ module SrlRuby
     # Now output the regexp literal
     root = ast_ptree.root
     options = root.is_a?(Regex::MatchOption) ? root.combine_opts : nil
-    return Regexp.new(root.to_str, options)
+    Regexp.new(root.to_str, options)
   end
 end # module

data/spec/acceptance/support/rule_file_ast_builder.rb

@@ -46,7 +46,7 @@ module Acceptance
         end
       end
 
-      return rule_file
+      rule_file
     end
 
     # rule('srl_heading' => %w[SRL: SRL_SOURCE]).as 'srl_source'

data/spec/acceptance/support/rule_file_parser.rb

@@ -14,7 +14,7 @@ module Acceptance # This module is used as a namespace
       File.open(filename, 'r') { |f| source = f.read }
       return source if source.nil? || source.empty?
 
-      return parse(source)
+      parse(source)
     end
 
     # Parse the rule file
@@ -41,8 +41,7 @@ module Acceptance # This module is used as a namespace
       ast_ptree = engine.convert(result)
 
       # Now output the regexp literal
-      root = ast_ptree.root
-      return root
+      ast_ptree.root
     end
   end
 end # module

data/spec/acceptance/support/rule_file_tokenizer.rb

@@ -58,7 +58,7 @@ module Acceptance
         tok_sequence << token unless token.nil?
       end
 
-      return tok_sequence
+      tok_sequence
     end
 
     private
@@ -68,13 +68,11 @@ module Acceptance
       curr_ch = scanner.peek(1)
       return nil if curr_ch.nil? || curr_ch.empty?
 
-      token = if state == :default
+      if state == :default
         default_mode
       else
         expecting_srl
       end
-
-      return token
     end
 
     def default_mode
@@ -103,7 +101,7 @@ module Acceptance
         raise ScanError, "Unknown token #{erroneous}"
       end
 
-      return token
+      token
     end
 
     def expecting_srl
@@ -123,7 +121,7 @@ module Acceptance
         raise e
       end
 
-      return token
+      token
     end
 
     def skip_noise
@@ -152,7 +150,7 @@ module Acceptance
       end
 
       curr_pos = scanner.pos
-      return curr_pos != pre_pos
+      curr_pos != pre_pos
     end
 
     def skip_comment

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: srl_ruby
 version: !ruby/object:Gem::Version
-  version: 0.4.11
+  version: 0.4.12
 platform: ruby
 authors:
 - Dimitri Geshef
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-04-17 00:00:00.000000000 Z
+date: 2022-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rley