opal 1.8.1 → 1.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8af8b2b4d485065e48d45e66e00075cf3b2f3d58c358355f5cbf5c97c46cd607
-  data.tar.gz: 87b7e4d8443eeb45953d48ca6ebb321b27269d9e7bbe2c4770c5189bcd44bcf9
+  metadata.gz: b8f2fff5a3c3229c43f5390af4758bb523362ea6ee1a1df8d2d90c53d502b575
+  data.tar.gz: 70966774cb6f97e29ed4b8516831ea94492c6f1cb83d42493a3ed88d146956a9
 SHA512:
-  metadata.gz: 615891333955fa65566ccad7f18d7a2a65905076db6268775bb5e9f06604b56e4ebfcb8fc05d181353c1bcfbba64c195c2eb4f71b37f805e9f7984f990e56a41
-  data.tar.gz: 1566eb09bf04acbcb49c7386ab9259cd13f8d94810e9dfecadf109b57ad2eaf479d1df419359a4a776b11aba2c7cc97d7df761c9ade46a2b8512d2718503ab69
+  metadata.gz: 693fe7106ef50e1bda17b92c8d548e5cd8419cfce0a4b5f2bde7accc87c905b0e407c5dd9ade1ff47249d0ffa2b8ca3d7fcbe2a12b5d1a2e45453a8a3400c72f
+  data.tar.gz: 22958073241a8d912931f9f3ce5c2d05674d82e74a2952f60f28664474ddc58fdf4639778ff3ac9cc67e9040ed8496aac91ab49c7de3d44a26f9687a33c96ba5

data/CHANGELOG.md

@@ -16,7 +16,7 @@ Changes are grouped as follows:
 
 
 
-## [1.8.1](https://github.com/opal/opal/compare/v1.8.0...v1.8.1) - 2023-11-09
+## [1.8.2](https://github.com/opal/opal/compare/v1.8.1...v1.8.2) - 2023-11-23
 
 
 <!--
@@ -25,9 +25,21 @@ Changes are grouped as follows:
 ### Added
 ### Removed
 ### Deprecated
+-->
+
 ### Performance
+
+- Optimize `Hash#rehash` for the common case, avoid calling `$slice` when no hash collision is present ([#2571](https://github.com/opal/opal/pull/2571))
+
 ### Fixed
--->
+
+- `String#{r,l,}strip`: Make them work like in MRI for non-breaking white-space ([#2612](https://github.com/opal/opal/pull/2612))
+- Compat regression fix: `Hash#to_n` should return a JS object ([#2613](https://github.com/opal/opal/pull/2613))
+
+
+
+
+## [1.8.1](https://github.com/opal/opal/compare/v1.8.0...v1.8.1) - 2023-11-09
 
 
 ### Fixed

data/lib/opal/version.rb

@@ -3,5 +3,5 @@
 module Opal
   # WHEN RELEASING:
   # Remember to update RUBY_ENGINE_VERSION in opal/corelib/constants.rb too!
-  VERSION = '1.8.1'
+  VERSION = '1.8.2'
 end

data/opal/corelib/constants.rb

@@ -1,8 +1,8 @@
 ::RUBY_PLATFORM       = 'opal'
 ::RUBY_ENGINE         = 'opal'
 ::RUBY_VERSION        = '3.2.0'
-::RUBY_ENGINE_VERSION = '1.8.1'
-::RUBY_RELEASE_DATE   = '2023-11-09'
+::RUBY_ENGINE_VERSION = '1.8.2'
+::RUBY_RELEASE_DATE   = '2023-11-23'
 ::RUBY_PATCHLEVEL     = 0
 ::RUBY_REVISION       = '0'
 ::RUBY_COPYRIGHT      = 'opal - Copyright (C) 2011-2023 Adam Beynon and the Opal contributors'

data/opal/corelib/runtime.js

@@ -2403,7 +2403,7 @@
       }
 
       var objects = keys.get(key_hash),
-          objects_copy = $slice(objects),
+          objects_copy = (objects.length === 1) ? objects : $slice(objects),
           object;
 
       for (var i=0; i<objects_copy.length; i++) {

data/opal/corelib/string.rb

@@ -724,7 +724,7 @@ class ::String < `String`
   end
 
   def lstrip
-    `self.replace(/^[\u0000\s]*/, '')`
+    `self.replace(/^[\x00\x09\x0a-\x0d\x20]*/, '')`
   end
 
   def ascii_only?
@@ -1023,7 +1023,7 @@ class ::String < `String`
   end
 
   def rstrip
-    `self.replace(/[\s\u0000]*$/, '')`
+    `self.replace(/[\x00\x09\x0a-\x0d\x20]*$/, '')`
   end
 
   def scan(pattern, no_matchdata: false, &block)
@@ -1201,7 +1201,7 @@ class ::String < `String`
   end
 
   def strip
-    `self.replace(/^[\s\u0000]*|[\s\u0000]*$/g, '')`
+    `self.replace(/^[\x00\x09\x0a-\x0d\x20]*|[\x00\x09\x0a-\x0d\x20]*$/g, '')`
   end
 
   def sub(pattern, replacement = undefined, &block)

data/spec/opal/core/string_spec.rb

@@ -26,3 +26,38 @@ describe 'Encoding' do
     end
   end
 end
+
+describe 'strip methods' do
+  def strip_cases(before, after, method)
+    before = before ? 1 : 0
+    after = after ? 1 : 0
+
+    it 'strip spaces' do
+      "#{"  " * before}ABC#{"  " * after}".send(method).should == "ABC"
+    end
+
+    it 'strips NUL bytes' do
+      "#{"\0" * before}ABC#{"\0" * after}".send(method).should == "ABC"
+    end
+
+    it "doesn't strip NBSPs" do
+      "#{"\u{a0}" * before}ABC#{"\u{a0}" * after}".send(method).should != "ABC"
+    end
+
+    it "strips all other supported whitespace characters" do
+      "#{"\r\n\t\v\f" * before}ABC#{"\r\n\t\v\f" * after}".send(method).should == "ABC"
+    end
+  end
+
+  describe '#lstrip' do
+   strip_cases true, false, :lstrip
+  end
+
+  describe '#rstrip' do
+    strip_cases false, true, :rstrip
+  end
+
+  describe '#strip' do
+    strip_cases true, true, :strip
+  end
+end

data/spec/opal/stdlib/native/hash_spec.rb

@@ -82,14 +82,14 @@ describe Hash do
     it 'converts a hash with native objects as values' do
       obj = { 'a_key' => `{ key: 1 }` }
       native = obj.to_n
-      `#{native}.get('a_key').key`.should == 1
+      `#{native}.a_key.key`.should == 1
     end
 
     it 'passes Ruby objects that cannot be converted' do
       object = Object.new
       hash = { foo: object }
       native = hash.to_n
-      expect(`#{native}.get('foo')`).to eq object
+      expect(`#{native}.foo`).to eq object
     end
   end
 end

data/stdlib/native.rb

@@ -587,14 +587,14 @@ unless Hash.method_defined? :_initialize
       }
     end
 
-    # @return a JavaScript Map but calling #to_n on
+    # @return a JavaScript object, in turn also calling #to_n on
     # all keys and values.
     def to_n
       %x{
-        var result = new Map();
+        var result = {};
 
         Opal.hash_each(self, false, function(key, value) {
-          result.set(#{Native.try_convert(`key`, `key`)} , #{Native.try_convert(`value`, `value`)});
+          result[#{Native.try_convert(`key`, `key`)}] = #{Native.try_convert(`value`, `value`)};
           return [false, false];
         });
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: opal
 version: !ruby/object:Gem::Version
-  version: 1.8.1
+  version: 1.8.2
 platform: ruby
 authors:
 - Elia Schito
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-11-09 00:00:00.000000000 Z
+date: 2023-11-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ast
@@ -1230,10 +1230,10 @@ licenses:
 metadata:
   homepage_uri: https://opalrb.com/
   bug_tracker_uri: https://github.com/opal/opal/issues
-  changelog_uri: https://github.com/opal/opal/blob/v1.8.1/CHANGELOG.md
-  readme_uri: https://github.com/opal/opal/blob/v1.8.1/README.md
-  api_documentation_uri: http://opalrb.com/docs/api/v1.8.1/index.html
-  guides_uri: http://opalrb.com/docs/guides/v1.8.1/index.html
+  changelog_uri: https://github.com/opal/opal/blob/v1.8.2/CHANGELOG.md
+  readme_uri: https://github.com/opal/opal/blob/v1.8.2/README.md
+  api_documentation_uri: http://opalrb.com/docs/api/v1.8.2/index.html
+  guides_uri: http://opalrb.com/docs/guides/v1.8.2/index.html
   chat_uri: https://gitter.im/opal/opal
   source_code_uri: https://github.com/opal/opal
 post_install_message: