From 4c7f789e942e78ebd3a7e3bf458c6cbe2133d692 Mon Sep 17 00:00:00 2001 From: Alan Wu Date: Tue, 29 Oct 2019 20:08:01 -0400 Subject: Allow only one argument for keyword_init struct ``` irb(main):001:0> RUBY_VERSION => "2.6.5" irb(main):002:0> S = Struct.new(:foo, keyword_init: true) => S(keyword_init: true) irb(main):003:0> S.new({foo: 23424}, 234) # I don't think this is intentional => # irb(main):004:0> ``` Tightening this up should inform users when they are confused about whether a struct is `keyword_init`. --- struct.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'struct.c') diff --git a/struct.c b/struct.c index 8294e9fa21..273c719aec 100644 --- a/struct.c +++ b/struct.c @@ -633,7 +633,7 @@ rb_struct_initialize_m(int argc, const VALUE *argv, VALUE self) n = num_members(klass); if (argc > 0 && RTEST(rb_struct_s_keyword_init(klass))) { struct struct_hash_set_arg arg; - if (argc > 2 || !RB_TYPE_P(argv[0], T_HASH)) { + if (argc > 1 || !RB_TYPE_P(argv[0], T_HASH)) { rb_raise(rb_eArgError, "wrong number of arguments (given %d, expected 0)", argc); } rb_mem_clear((VALUE *)RSTRUCT_CONST_PTR(self), n); -- cgit v1.2.3