Fix formatting and add Changelog entry

Author: andrykonchin

CHANGELOG.md

@@ -83,6 +83,7 @@ Compatibility:
 * Modify `Struct#{inspect,to_s}` to match MRI when the struct is nested inside of an anonymous class or module (@st0012, @nirvdrum).
 * `Fiber.current` and `Fiber#transfer` are available without `require 'fiber'` like in CRuby 3.1 (#2733, @eregon).
 * Add `freeze` keyword argument to `Marshal.load` (#2733, @andrykonchin).
+* Add `StructClass#keyword_init?` method (#2377, @moste00).
 
 Performance:
 

spec/ruby/core/struct/keyword_init_spec.rb

@@ -19,21 +19,21 @@
     end
 
     it "returns nil for a struct that does specify keyword_init to be nil" do
-      struct = Struct.new(:arg,keyword_init: nil)
+      struct = Struct.new(:arg, keyword_init: nil)
       struct.keyword_init?.should be_nil
     end
 
-    it "returns true or false for truthy values in general, not just true" do
-      struct = Struct.new(:arg,keyword_init: 1)
+    it "returns true for any truthy value, not just for true" do
+      struct = Struct.new(:arg, keyword_init: 1)
       struct.keyword_init?.should be_true
 
-      struct = Struct.new(:arg,keyword_init: "")
+      struct = Struct.new(:arg, keyword_init: "")
       struct.keyword_init?.should be_true
 
-      struct = Struct.new(:arg,keyword_init: [])
+      struct = Struct.new(:arg, keyword_init: [])
       struct.keyword_init?.should be_true
 
-      struct = Struct.new(:arg,keyword_init: {})
+      struct = Struct.new(:arg, keyword_init: {})
       struct.keyword_init?.should be_true
     end
   end

src/main/ruby/truffleruby/core/struct.rb

@@ -94,13 +94,10 @@ def self.inspect
       const_set :KEYWORD_INIT, keyword_init
 
       def self.keyword_init?
-        kw_init = self.const_get(:KEYWORD_INIT)
-        return nil if Primitive.nil?(kw_init)
-
-        Primitive.as_boolean(kw_init)
-      end
-
+        return nil if Primitive.nil?(self::KEYWORD_INIT)
+        Primitive.as_boolean(self::KEYWORD_INIT)
       end
+    end
 
     const_set klass_name, klass if klass_name
 

test/mri/excludes/TestStruct/SubStruct.rb

@@ -5,4 +5,3 @@
 exclude :test_struct_new_with_empty_hash, "needs investigation"
 exclude :test_parameters, "<[[:req, :_]]> expected but was"
 exclude :test_keyword_args_warning, "<{:a=>1}> expected but was"
-exclude :test_struct_keyword_init_p, "NoMethodError: undefined method `keyword_init?' for #<Class:0x6bf8>(keyword_init: true)"

test/mri/excludes/TestStruct/TopStruct.rb

@@ -3,6 +3,5 @@
 exclude :test_nonascii, "needs investigation"
 exclude :test_redefinition_warning, "needs investigation"
 exclude :test_struct_new_with_empty_hash, "needs investigation"
-exclude :test_struct_keyword_init_p, "NoMethodError: undefined method `keyword_init?' for #<Class:0xc3f888>(keyword_init: true)"
 exclude :test_parameters, "<[[:req, :_]]> expected but was"
 exclude :test_keyword_args_warning, "expected: /warning: Passing only keyword arguments to Struct#initialize will behave differently from Ruby 3\\.2\\./"