loxxy 0.4.09 → 0.4.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ece684c97fd1a382d69d7fb821bd33ca29b662c07230e1f305a9fb50e4abef1
-  data.tar.gz: 0400c4fe48329686f63d34693658c6a8960759ba10173958e0ef0c6b2b75880b
+  metadata.gz: 21a1cbd9499c50fc448238c4b46fabf32add037e813d073df2e88126682076cb
+  data.tar.gz: 54b00da85e3b48be9506ad9f5c52e4844c26491dcf3007a360af2c5a925296d7
 SHA512:
-  metadata.gz: 373db901360190ef0cd84fa3142ec7705922a41d77ec36e3a2f6b8a0b74fbc27c6f0e6de00fc8c38122813a4333758a497b9a88fc372c977a558c8722c19dccb
-  data.tar.gz: a52c9d016e7761ce6b0415cf7c5a409a90106e96c8c18dd8d7e264f87bf43a293d7314bf3854164e809cc1c9cb8354e4c9b97d5d1f2f57d6730e0ecf229e4502
+  metadata.gz: a35a3aefa589ba7d2d5d0c3e2f9b21b837855ea46e93817e890649d5a0a8fc7f0d02b3fd2fdf0072c0f2b7ccdd4ad6c77e4c585f7dd08fe6394ee5f08dca23af
+  data.tar.gz: e5d91111f82398021791ef5e6c3f0625e3b16527aec81948dc730381b6da882a86ec7c6a11fd25273c2b8e06efb781421ee2de9b586833f0eb234cbbfcc67982

data/.rubocop.yml

@@ -2,10 +2,18 @@ AllCops:
   Exclude:
     - 'exp/**/*'
     - 'demo/**/*'
+  NewCops: enable
+  SuggestExtensions: false
 
 Gemspec/DeprecatedAttributeAssignment:
   Enabled: true
 
+Gemspec/DevelopmentDependencies:
+  EnforcedStyle: gemspec
+
+Gemspec/RequireMFA: # new in 1.23
+  Enabled: true
+
 Layout/ArgumentAlignment:
   Enabled: false
 
@@ -60,117 +68,12 @@ Layout/MultilineMethodCallBraceLayout:
   Enabled: true
   EnforcedStyle: same_line
 
-Layout/SpaceAroundOperators:
-  Enabled: true
-
-Layout/SpaceBeforeBrackets:
-  Enabled: true
-
-Layout/SpaceInsideParens:
-  Enabled: true
-
-Layout/IndentationStyle:
-  Enabled: true
-
-Layout/SpaceAroundMethodCallOperator:
-  Enabled: true
-
-Layout/TrailingEmptyLines:
-  Enabled: true
-
-Layout/TrailingWhitespace:
-  Enabled: true
-
-Lint/AmbiguousAssignment:
-  Enabled: true
-
 Lint/AmbiguousOperatorPrecedence: # new in 1.21
   Enabled: true
 
-Lint/AmbiguousRange:
-  Enabled: true
-
-Lint/DeprecatedConstants:
-  Enabled: true
-
-Lint/DuplicateBranch:
-  Enabled: true
-
-Lint/DuplicateRegexpCharacterClassElement:
-  Enabled: true
-
-Lint/EmptyBlock:
-  Enabled: true
-
 Lint/EmptyClass:
   Enabled: false
 
-Lint/EmptyInPattern:
-  Enabled: true
-
-Lint/IncompatibleIoSelectWithFiberScheduler: # new in 1.21
-  Enabled: true
-
-Lint/LambdaWithoutLiteralBlock:
-  Enabled: true
-
-Lint/Loop:
-  Enabled: true
-
-Lint/NoReturnInBeginEndBlocks:
-  Enabled: true
-
-Lint/NumberedParameterAssignment:
-  Enabled: true
-
-Lint/OrAssignmentToConstant:
-  Enabled: true
-
-Lint/RaiseException:
-  Enabled: true
-
-Lint/RedundantDirGlobSort:
-  Enabled: true
-
-Lint/RequireRelativeSelfPath: # new in 1.22
-  Enabled: true
-
-Lint/RescueException:
-  Enabled: true
-
-Lint/StructNewOverride:
-  Enabled: true
-
-Lint/SymbolConversion:
-  Enabled: true
-
-Lint/ToEnumArguments:
-  Enabled: true
-
-Lint/TripleQuotes:
-  Enabled: true
-
-Lint/UnexpectedBlockArity:
-  Enabled: true
-
-Lint/UnmodifiedReduceAccumulator:
-  Enabled: true
-
-Lint/UnusedMethodArgument:
-  Enabled: true
-
-Lint/UselessAccessModifier:
-  Enabled: true
-
-Lint/UselessRuby2Keywords: # new in 1.23
-  Enabled: true
-
-Lint/Void:
-  Enabled: false
-
-Lint/UselessAssignment:
-  Enabled: true
-
 Metrics/AbcSize:
   Enabled: false
 
@@ -204,21 +107,12 @@ Metrics/ModuleLength:
 Metrics/PerceivedComplexity:
   Enabled: false
 
-Naming/BlockForwarding: # new in 1.24
-  Enabled: true
-
 Naming/ConstantName:
   Enabled: false
 
 Naming/ClassAndModuleCamelCase:
   Enabled: false
 
-Naming/BlockParameterName:
-  Enabled: true
-
-Naming/InclusiveLanguage:
-  Enabled: true
-
 Naming/MethodParameterName:
   Enabled: false
 
@@ -228,15 +122,6 @@ Naming/MethodName:
 Naming/VariableName:
   Enabled: false
 
-Security/IoMethods: # new in 1.22
-  Enabled: true
-
-Style/Alias:
-  Enabled: true
-
-Style/ArgumentsForwarding:
-  Enabled: true
-
 Style/AsciiComments:
   Enabled: false
 
@@ -255,9 +140,6 @@ Style/ClassCheck:
 Style/ClassVars:
   Enabled: false
 
-Style/CollectionCompact:
-  Enabled: true
-
 Style/ColonMethodCall:
   Enabled: false
 
@@ -270,223 +152,69 @@ Style/CommentedKeyword:
 Style/ConditionalAssignment:
   Enabled: false
 
-Style/DefWithParentheses:
-  Enabled: true
-
 Style/Documentation:
   Enabled: false
 
-Style/DocumentDynamicEvalDefinition:
-  Enabled: true
-
 Style/ExpandPathArguments:
   Enabled: false
 
-Style/ExponentialNotation:
-  Enabled: true
-
 Style/GuardClause:
   Enabled: false
 
-Style/HashEachMethods:
-  Enabled: true
-
-Style/HashExcept:
-  Enabled: true
-
-Style/HashTransformKeys:
-  Enabled: true
-
-Style/HashTransformValues:
-  Enabled: true
-
 Style/IfUnlessModifier:
   Enabled: false
 
-Style/InPatternThen:
-  Enabled: true
-
 Style/InverseMethods:
   Enabled: false
 
 Style/MissingRespondToMissing:
   Enabled: false
 
-Style/MultilineInPatternThen:
-  Enabled: true
-
-Style/NegatedIfElseCondition:
-  Enabled: true
-
 Style/Next:
   Enabled: false
 
-Style/NilLambda:
-  Enabled: true
-
-Style/NumberedParameters: # new in 1.22
-  Enabled: true
-
-Style/NumberedParametersLimit: # new in 1.22
-  Enabled: true
-
 Style/NumericLiterals:
   Enabled: false
 
-Style/QuotedSymbols:
-  Enabled: true
-
-Style/RaiseArgs:
-  Enabled: true
-
-Style/RedundantArgument:
-  Enabled: true
-
 Style/RedundantReturn:
   Enabled: false
 
-Style/RedundantSelf:
-  Enabled: true
-
-Style/RedundantSelfAssignmentBranch:
-  Enabled: true
-
 Style/RegexpLiteral:
   Enabled: false
 
 Style/PercentLiteralDelimiters:
   Enabled: false
 
-Style/SelectByRegexp: # new in 1.22
-  Enabled: true
-
 Style/StderrPuts:
   Enabled: false
 
-Style/StringLiterals:
-  Enabled: true
-
-Style/SwapValues:
-  Enabled: true
-
 Style/TernaryParentheses:
   Enabled: false
 
 Style/UnlessElse:
   Enabled: false
 
-# Rubocop complains when it doesn't find an explicit setting for the following cops:
-Layout/EmptyLinesAroundAttributeAccessor:
-  Enabled: true
-
-Lint/BinaryOperatorWithIdenticalOperands:
-  Enabled: true
-
-Lint/DeprecatedOpenSSLConstant:
-  Enabled: true
-
-Lint/DuplicateElsifCondition:
-  Enabled: true
-
-Lint/DuplicateRescueException:
-  Enabled: true
-
-Lint/EmptyConditionalBody:
-  Enabled: true
-
-Lint/FloatComparison:
-  Enabled: true
-
-Lint/MissingSuper:
-  Enabled: true
-
-Lint/MixedRegexpCaptureTypes:
-  Enabled: true
-
-Lint/OutOfRangeRegexpRef:
-  Enabled: true
-
-Lint/SelfAssignment:
-  Enabled: true
-
-Lint/TopLevelReturnWithArgument:
-  Enabled: true
-
-Lint/UnreachableLoop:
-  Enabled: true
-
-Style/AccessorGrouping:
-  Enabled: true
-
-Style/ArrayCoercion:
-  Enabled: true
-
-Style/BisectedAttrAccessor:
-  Enabled: true
-
-Style/CaseLikeIf:
-  Enabled: true
-
-Style/EndlessMethod:
-  Enabled: true
-
-Style/ExplicitBlockArgument:
-  Enabled: true
-
-Style/FileRead: # new in 1.24
-  Enabled: true
-
-Style/FileWrite: # new in 1.24
-  Enabled: true
-
-Style/GlobalStdStream:
-  Enabled: true
-
-Style/HashAsLastArrayItem:
-  Enabled: true
-
-Style/HashConversion:
-  Enabled: true
-
-Style/HashLikeCase:
-  Enabled: true
-  
-Style/IfWithBooleanLiteralBranches:
-  Enabled: true
-  
-Style/MapToHash: # new in 1.24
-  Enabled: true 
-
-Style/OpenStructUse: # new in 1.23
-  Enabled: true  
-
-Style/OptionalBooleanParameter:
-  Enabled: true
-  
-
-Style/RedundantAssignment:
-  Enabled: true
-
-Style/RedundantFetchBlock:
-  Enabled: true
-
-Style/RedundantFileExtensionInRequire:
-  Enabled: true
-
-Style/RedundantRegexpCharacterClass:
-  Enabled: true
-
-Style/RedundantRegexpEscape:
-  Enabled: true
-
-Style/SingleArgumentDig:
-  Enabled: true
-
-Style/SlicingWithRange:
-  Enabled: true
-
-Style/StringChars:
-  Enabled: true
-
-Style/StringConcatenation:
-  Enabled: true
+## Optional rubocop-rspec plugin rule checking
+#RSpec/ContextWording:
+#  Enabled: false
+#
+#RSpec/DescribeClass:
+#  Enabled: false
+#
+#RSpec/ExampleLength:
+#  Max: 40
+#
+#RSpec/MultipleExpectations:
+#  Max: 25
+#
+#RSpec/NoExpectationExample:
+#  Enabled: false
+#
+#RSpec/RepeatedDescription:
+#  Enabled: false
+#
+#RSpec/RepeatedExample:
+#  Enabled: false
+#
+#RSpec/RepeatedExampleGroupDescription:
+#  Enabled: false

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## [0.4.10] - 2025-0x-xx
+- Tested with Rley 0.9.02 and Rubocop 1.74
+
 ## [0.4.09] - 2025-01-16
 - Tested with MRI Ruby 3.4.1.
 - Minor changes

data/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2020-2022 Dimitri Geshef
+Copyright (c) 2020-2025 Dimitri Geshef
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/lib/loxxy/ast/ast_builder.rb

@@ -47,7 +47,7 @@ module Loxxy
       # Create a new AST builder instance.
       # @param theTokens [Array<Rley::Lexical::Token>] The sequence of input tokens.
       def initialize(theTokens)
-        super(theTokens)
+        super
         @strict = false
       end
 

data/lib/loxxy/back_end/engine.rb

@@ -446,6 +446,7 @@ module Loxxy
         add_native_fun('print_error', native_print_error)
       end
 
+      # rubocop: disable Lint/UselessConstantScoping
       NativeFunction = Struct.new(:callable, :interp) do
         def accept(_visitor)
           interp.expr_stack.push self
@@ -459,6 +460,7 @@ module Loxxy
           '<native fn>'
         end
       end
+      # rubocop: enable Lint/UselessConstantScoping
 
       def add_native_fun(aName, aProc)
         native_fun = Variable.new(aName, NativeFunction.new(aProc, self))

data/lib/loxxy/back_end/environment.rb

@@ -49,7 +49,7 @@ module Loxxy
       # Returns a string with a human-readable representation of the object.
       # @return [String]
       def inspect
-        +"#<#{self.class}:#{object_id.to_s(16)}>"
+        "#<#{self.class}:#{object_id.to_s(16)}>"
       end
 
       private

data/lib/loxxy/front_end/scanner.rb

@@ -20,14 +20,14 @@ module Loxxy
     # Delimiters: e.g. parentheses '(',  ')'
     # Separators: e.g. comma
     class Scanner
-      PATT_BLOCK_COMMENT_BEGIN = /\/\*/.freeze
-      PATT_BLOCK_COMMENT_END = /\*\//.freeze
-      PATT_COMPARISON = /[!=><]=?/.freeze
-      PATT_IDENTIFIER = /[a-zA-Z_][a-zA-Z_0-9]*/.freeze
-      PATT_LINE_COMMENT = /\/\/[^\r\n]*/.freeze
-      PATT_NEWLINE = /(?:\r\n)|\r|\n/.freeze
-      PATT_NUMBER = /\d+(?:\.\d+)?/.freeze
-      PATT_WHITESPACE = /[ \t\f]+/.freeze
+      PATT_BLOCK_COMMENT_BEGIN = /\/\*/
+      PATT_BLOCK_COMMENT_END = /\*\//
+      PATT_COMPARISON = /[!=><]=?/
+      PATT_IDENTIFIER = /[a-zA-Z_][a-zA-Z_0-9]*/
+      PATT_LINE_COMMENT = /\/\/[^\r\n]*/
+      PATT_NEWLINE = /(?:\r\n)|\r|\n/
+      PATT_NUMBER = /\d+(?:\.\d+)?/
+      PATT_WHITESPACE = /[ \t\f]+/
 
       # @return [StringScanner] Low-level input scanner
       attr_reader(:scanner)
@@ -148,7 +148,7 @@ module Loxxy
               elsif (lexeme = scanner.scan(PATT_COMPARISON))
                 # One or two special character tokens
                 build_token(Lexeme2name[lexeme], lexeme)
-              elsif scanner.scan(/"/) # Start of string detected...
+              elsif scanner.scan('"') # Start of string detected...
                 build_string_token
               elsif (lexeme = scanner.scan(PATT_NUMBER))
                 build_token('NUMBER', lexeme)

data/lib/loxxy/interpreter.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require_relative './front_end/parser'
-require_relative './ast/ast_visitor'
-require_relative './back_end/engine'
+require_relative 'front_end/parser'
+require_relative 'ast/ast_visitor'
+require_relative 'back_end/engine'
 
 module Loxxy
   # A Lox tree-walking interpreter.

data/lib/loxxy/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Loxxy
-  VERSION = '0.4.09'
+  VERSION = '0.4.10'
 end

data/loxxy.gemspec

@@ -42,11 +42,13 @@ Gem::Specification.new do |spec|
   spec.description   = <<-DESCR_END
   A Ruby implementation of the Lox programming language. Lox is a dynamically typed,
   object-oriented programming language that features first-class functions, closures,
-  classes, and inheritance.
+  classes, and inheritance. The interpreter passes all unit tests from the original Java implementation.
+  It is also able to run `LoxLox` a Lox interpreter written in Lox itself.
   DESCR_END
   spec.homepage      = 'https://github.com/famished-tiger/loxxy'
   spec.license       = 'MIT'
-  spec.required_ruby_version = '>= 2.6'
+  spec.required_ruby_version = '>= 3.1'
+  spec.metadata = { 'rubygems_mfa_required' => 'true' }
 
   spec.bindir        = 'bin'
   spec.executables   = ['loxxy']
@@ -56,11 +58,10 @@ Gem::Specification.new do |spec|
   PkgExtending.pkg_documentation(spec)
 
   # Runtime dependencies
-  spec.add_dependency 'rley', '~> 0.8.10'
+  spec.add_dependency 'rley', '~> 0.9', '>= 0.9.02'
 
   # Development dependencies
-  spec.add_development_dependency 'bundler', '~> 2.5.0'
-  spec.add_development_dependency 'rake', '~> 13.1.0'
-  spec.add_development_dependency 'rspec', '~> 3.12'
-  spec.add_development_dependency 'yard', '~> 0.9.34'
+  spec.add_development_dependency 'bundler', '~> 2.4', '>= 2.4.1'
+  spec.add_development_dependency 'rake', '~> 13'
+  spec.add_development_dependency 'rspec', '~> 3', '>= 3.10.0'
 end

data/spec/back_end/engine_spec.rb

@@ -12,50 +12,52 @@ module Loxxy
       let(:sample_options) do
         { ostream: StringIO.new }
       end
-      subject { Engine.new(sample_options) }
+
+      subject(:engine) { described_class.new(sample_options) }
 
       context 'Initialization:' do
-        it 'should accept an option Hash at initialization' do
-          expect { Engine.new(sample_options) }.not_to raise_error
+        it 'accepts an option Hash at initialization' do
+          expect { described_class.new(sample_options) }.not_to raise_error
         end
 
-        it 'should know its config options' do
-          expect(subject.config).to eq(sample_options)
+        it 'knows its config options' do
+          expect(engine.config).to eq(sample_options)
         end
 
-        it 'should have an empty stack' do
-          expect(subject.stack).to be_empty
+        it 'has an empty stack' do
+          expect(engine.stack).to be_empty
         end
       end
 
       context 'Listening to visitor events:' do
         let(:greeting) { Datatype::LXString.new('Hello, world') }
-        let(:sample_pos) { double('fake-position') }
+        # let(:sample_pos) { double('fake-position') }
+        let(:sample_pos) { instance_double(Rley::Lexical::Position) }
         let(:var_decl) { Ast::LoxVarStmt.new(sample_pos, 'greeting', greeting) }
         let(:lit_expr) { Ast::LoxLiteralExpr.new(sample_pos, greeting) }
 
-        it "should react to 'after_var_stmt' event" do
+        it "reacts to 'after_var_stmt' event" do
           # Precondition: value to assign is on top of expr stack
-          subject.expr_stack.push(greeting)
+          engine.expr_stack.push(greeting)
 
-          expect { subject.after_var_stmt(var_decl) }.not_to raise_error
-          current_env = subject.symbol_table.current_env
-          expect(current_env.defns['greeting']).to be_kind_of(Variable)
+          expect { engine.after_var_stmt(var_decl) }.not_to raise_error
+          current_env = engine.symbol_table.current_env
+          expect(current_env.defns['greeting']).to be_a(Variable)
           expect(current_env.defns['greeting'].value).to eq(greeting)
         end
 
-        it "should react to 'before_literal_expr' event" do
-          expect { subject.before_literal_expr(lit_expr) }.not_to raise_error
-          expect(subject.expr_stack.pop).to eq(greeting)
+        it "reacts to 'before_literal_expr' event" do
+          expect { engine.before_literal_expr(lit_expr) }.not_to raise_error
+          expect(engine.expr_stack.pop).to eq(greeting)
         end
       end
 
       context 'Built-in functions:' do
-        it 'should provide built-in functions' do
-          symb_table = subject.symbol_table
+        it 'provides built-in functions' do
+          symb_table = engine.symbol_table
           %w[clock getc chr exit print_error].each do |name|
             fun_var = symb_table.current_env.defns[name]
-            expect(fun_var.value).to be_kind_of(BackEnd::Engine::NativeFunction)
+            expect(fun_var.value).to be_a(BackEnd::Engine::NativeFunction)
           end
         end
       end