fOOrth 0.6.5 → 0.6.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c3e8b4a5ccbb3c56347525737ec50f95a1016e19
-  data.tar.gz: 9bdb8b038c70895ddb82722ff0deeed11c3bdf47
+  metadata.gz: b78d9f4016b73209d62eaf5c8d01497f64b65582
+  data.tar.gz: 98ec2f26ae3bf20ea9a65d120ecb0d9bc6b2f88c
 SHA512:
-  metadata.gz: 8fe86ae93e915c05e9702fd3996d96a258cd0c5e164292eb2c9a1776a576a2c2878fd59c640bdae12e0543689cf3854b2a24076e2c2b8534b6cc4e6180d56241
-  data.tar.gz: 04f4a12816c42a8b698a2f7c43055d7ba3ce3151dee06103f1d17826cc9182355f66240a3d8390b6939376dcd193abad00a4271507d7bcf2b5cac7b2f1fa0d18
+  metadata.gz: f04893f0850026b62865f1087298d8f8ad1a621da78d097a96106adcba71f00ba14abe1968992bcdda57ef14ed6a0bfc4969d9f3b291793b55618d0b7cad0e09
+  data.tar.gz: 0f17fc2d2d9dd61a37e6b3be5440346a18af935d591d261ef53a3d7b2eb9e94ef29460bf6b40babe243d39efb4139dc63bd97081cb098bac012a04ac03869dc6

data/integration/data_ref_lib_tests.rb

@@ -22,6 +22,16 @@ class DataRefLibraryTester < Minitest::Test
 
     foorth_equal('10 val#: #test2',   [])
     foorth_equal('#test2',            [10])
+
+    foorth_run(': test_tv3 42 val#: #ttv3 ; ')
+    foorth_equal('#ttv3',             [nil])
+    foorth_run('test_tv3')
+    foorth_equal('#ttv3',             [42])
+
+    foorth_run(': test_tv5 42 var#: #ttv5 ; ')
+    foorth_raises('#ttv5 #')
+    foorth_run('test_tv5')
+    foorth_equal('#ttv5 @',           [42])
   end
 
   def test_thread_vars_some_more
@@ -39,6 +49,16 @@ class DataRefLibraryTester < Minitest::Test
 
     foorth_equal('10 val$: $test2',   [])
     foorth_equal('$test2',            [10])
+
+    foorth_run(': test_gv4 69 val$: $tgv4 ; ')
+    foorth_equal('$tgv4',             [nil])
+    foorth_run('test_gv4')
+    foorth_equal('$tgv4',             [69])
+
+    foorth_run(': test_gv6 69 var$: $tgv6 ; ')
+    foorth_raises('$tgv6 @')
+    foorth_run('test_gv6')
+    foorth_equal('$tgv6 @',           [69])
   end
 
   def test_shared_instance_variables

data/lib/fOOrth/debug/display_abort.rb

@@ -4,8 +4,6 @@
 module XfOOrth
 
   #* debug/display_abort.rb - Display diagnostic information on an error.
-  #<br>Endemic Code Smells
-  #* :reek:TooManyStatements
   class VirtualMachine
 
     #Display the diagnostic data required for a language abort error.

data/lib/fOOrth/library/data_ref_library.rb

@@ -31,45 +31,41 @@ module XfOOrth
 
   # Thread Variables
   # [n] var#: #tv [], Thread.current[#tv] = [n]
-  VirtualMachine.create_shared_method('var#:', VmSpec, [], &lambda {|vm|
+  VirtualMachine.create_shared_method('var#:', VmSpec, [:immediate], &lambda {|vm|
     name   = vm.parser.get_word()
     error "F10: Invalid var name #{name}" unless /^#[a-z][a-z0-9_]*$/ =~ name
     symbol = XfOOrth::SymbolMap.add_entry(name)
-    @data[symbol] = [vm.pop]
-
+    process_text("vm.data[#{symbol.inspect}] = [vm.pop]; ")
     vm.create_exclusive_method(name, ThreadVarSpec, [])
   })
 
   # Thread values.
   # [n] val#: #tv [], Thread.current[#tv] = n
-  VirtualMachine.create_shared_method('val#:', VmSpec, [], &lambda {|vm|
+  VirtualMachine.create_shared_method('val#:', VmSpec, [:immediate], &lambda {|vm|
     name   = vm.parser.get_word()
     error "F10: Invalid val name #{name}" unless /^#[a-z][a-z0-9_]*$/ =~ name
     symbol = XfOOrth::SymbolMap.add_entry(name)
-    @data[symbol] = vm.pop
-
+    process_text("vm.data[#{symbol.inspect}] = vm.pop; ")
     vm.create_exclusive_method(name, ThreadVarSpec, [])
   })
 
   # Global Variables
   # [n] var$: $gv [], $gv = [n]
-  VirtualMachine.create_shared_method('var$:', VmSpec, [], &lambda {|vm|
+  VirtualMachine.create_shared_method('var$:', VmSpec, [:immediate], &lambda {|vm|
     name   = vm.parser.get_word()
     error "F10: Invalid var name #{name}" unless /^\$[a-z][a-z0-9_]*$/ =~ name
     symbol = XfOOrth::SymbolMap.add_entry(name)
-    eval "#{'$' + symbol.to_s} = [vm.pop]"
-
+    process_text("#{'$' + symbol.to_s} = [vm.pop]; ")
     $FOORTH_GLOBALS[symbol] = GlobalVarSpec.new(name, symbol, [])
   })
 
   # Global Values
   # [n] val$: $gv [], $gv = n
-  VirtualMachine.create_shared_method('val$:', VmSpec, [], &lambda {|vm|
+  VirtualMachine.create_shared_method('val$:', VmSpec, [:immediate], &lambda {|vm|
     name   = vm.parser.get_word()
     error "F10: Invalid val name #{name}" unless /^\$[a-z][a-z0-9_]*$/ =~ name
     symbol = XfOOrth::SymbolMap.add_entry(name)
-    eval "#{'$' + symbol.to_s} = vm.pop"
-
+    process_text("#{'$' + symbol.to_s} = vm.pop; ")
     $FOORTH_GLOBALS[symbol] = GlobalVarSpec.new(name, symbol, [])
   })
 

data/lib/fOOrth/version.rb

@@ -3,5 +3,5 @@
 #* version.rb - The version string for fOOrth.
 module XfOOrth
   #The version string for fOOrth.
-  VERSION = "0.6.5"
+  VERSION = "0.6.6"
 end

data/reek.txt

@@ -1 +1,59 @@
-0 total warnings
+lib/fOOrth/compiler/context/map_name.rb -- 1 warning:
+  [31]:ControlParameter: XfOOrth::Context#do_map_name is controlled by argument allow_defaults [https://github.com/troessner/reek/blob/master/docs/Control-Parameter.md]
+lib/fOOrth/library/compile_library.rb -- 2 warnings:
+  [230]:ControlParameter: XfOOrth#self.name_to_type is controlled by argument cast_spec [https://github.com/troessner/reek/blob/master/docs/Control-Parameter.md]
+  [191]:TooManyStatements: XfOOrth#self.add_common_compiler_locals has approx 8 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+lib/fOOrth/library/formatting/bullets.rb -- 10 warnings:
+  [20, 22]:DuplicateMethodCall: XfOOrth::BulletPoints#add calls raw_bullet.in_array 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [62, 68]:DuplicateMethodCall: XfOOrth::BulletPoints#render_bullet calls ' ' * @key_length 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [67, 76]:DuplicateMethodCall: XfOOrth::BulletPoints#render_bullet calls result << temp 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [63, 69]:DuplicateMethodCall: XfOOrth::BulletPoints#render_bullet calls temp.length 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [26, 27]:FeatureEnvy: XfOOrth::BulletPoints#add refers to items more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [20, 22]:FeatureEnvy: XfOOrth::BulletPoints#add refers to raw_bullet more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [18, 23]:FeatureEnvy: XfOOrth::BulletPoints#add refers to raw_item more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [83]:IrresponsibleModule: Array has no descriptive comment [https://github.com/troessner/reek/blob/master/docs/Irresponsible-Module.md]
+  [103]:IrresponsibleModule: Hash has no descriptive comment [https://github.com/troessner/reek/blob/master/docs/Irresponsible-Module.md]
+  [51]:TooManyStatements: XfOOrth::BulletPoints#render_bullet has approx 15 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+lib/fOOrth/library/formatting/columns.rb -- 2 warnings:
+  [36]:NestedIterators: XfOOrth::ColumnizedPage#render contains iterators nested 2 deep [https://github.com/troessner/reek/blob/master/docs/Nested-Iterators.md]
+  [31]:TooManyStatements: XfOOrth::ColumnizedPage#render has approx 8 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+lib/fOOrth/library/introspection/class.rb -- 15 warnings:
+  [35, 42]:DuplicateMethodCall: Class#foorth_method_info calls !spec.has_tag?(:stub) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [36, 43]:DuplicateMethodCall: Class#foorth_method_info calls (results << ["", ""]).concat(info) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [36, 43]:DuplicateMethodCall: Class#foorth_method_info calls (results << ["", ""]).concat(info).concat(spec.get_info) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [36, 43]:DuplicateMethodCall: Class#foorth_method_info calls results << ["", ""] 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [36, 43]:DuplicateMethodCall: Class#foorth_method_info calls spec.get_info 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [35, 42]:DuplicateMethodCall: Class#foorth_method_info calls spec.has_tag?(:stub) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [62, 74]:DuplicateMethodCall: Class#get_info calls (results << ["", ""]).concat(info) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [61, 73]:DuplicateMethodCall: Class#get_info calls XfOOrth::SymbolMap.map_info(name) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [62, 74]:DuplicateMethodCall: Class#get_info calls results << ["", ""] 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [65, 77]:DuplicateMethodCall: Class#get_info calls results.concat(info) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [65, 77]:DuplicateMethodCall: Class#get_info calls results.concat(info).concat(spec.get_info) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [65, 77]:DuplicateMethodCall: Class#get_info calls spec.get_info 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [35, 36, 42, 43]:FeatureEnvy: Class#foorth_method_info refers to spec more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [28]:TooManyStatements: Class#foorth_method_info has approx 10 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+  [54]:TooManyStatements: Class#get_info has approx 14 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+lib/fOOrth/library/introspection/context.rb -- 3 warnings:
+  [14, 28]:DuplicateMethodCall: XfOOrth::Context#get_info calls results << ["", ""] 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [14, 17, 18, 19, 21, 22, 28, 29]:FeatureEnvy: XfOOrth::Context#get_info refers to results more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [10]:TooManyStatements: XfOOrth::Context#get_info has approx 11 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+lib/fOOrth/library/introspection/object.rb -- 3 warnings:
+  [53, 57]:FeatureEnvy: Object#foorth_method_info refers to results more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [52, 53]:FeatureEnvy: Object#foorth_method_info refers to spec more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [18]:TooManyStatements: Object#get_info has approx 12 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+lib/fOOrth/library/introspection/string.rb -- 10 warnings:
+  [17, 20]:DuplicateMethodCall: String#foorth_method_scan calls (results << ["", ""]).concat(info) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [17, 20]:DuplicateMethodCall: String#foorth_method_scan calls (results << ["", ""]).concat(info).concat(spec.get_info) 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [17, 20]:DuplicateMethodCall: String#foorth_method_scan calls results << ["", ""] 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [17, 20]:DuplicateMethodCall: String#foorth_method_scan calls spec.get_info 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+  [17, 20, 23]:FeatureEnvy: String#foorth_method_scan refers to found more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [17, 20, 23]:FeatureEnvy: String#foorth_method_scan refers to results more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [13, 17, 20]:FeatureEnvy: String#foorth_method_scan refers to spec more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+  [7]:TooManyStatements: String#foorth_method_scan has approx 12 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+  [14]:UncommunicativeVariableName: String#foorth_method_scan has the variable name 'a' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+  [14]:UncommunicativeVariableName: String#foorth_method_scan has the variable name 'b' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+lib/fOOrth/library/introspection/vm.rb -- 3 warnings:
+  [10]:TooManyStatements: XfOOrth::VirtualMachine#get_info has approx 22 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+  [47]:UncommunicativeVariableName: XfOOrth::VirtualMachine#get_info has the variable name 'a' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+  [47]:UncommunicativeVariableName: XfOOrth::VirtualMachine#get_info has the variable name 'b' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+49 total warnings

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fOOrth
 version: !ruby/object:Gem::Version
-  version: 0.6.5
+  version: 0.6.6
 platform: ruby
 authors:
 - Peter Camilleri
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-14 00:00:00.000000000 Z
+date: 2016-09-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler