From: Yukihiro Matsumoto <matz@...>
Date: 2005-02-10T00:53:27+09:00
Subject: Re: BUG: Struct.new(:a?).instance_methods

Hi

In message "Re: BUG: Struct.new(:a?).instance_methods"
    on Wed, 9 Feb 2005 23:15:01 +0900, "Cs. Henk" <csaba-ml@beastie.creo.hu> writes:

|Hi, getting an ArgumentError with "NULL pointer given" doesn't seem to
|be the proper reaction upon doing what's in the subject.

Right.  "a?" should not be allowed for a struct member name, since
there cannot be a "a?=" method.  Here's the patch.

							matz.
--- struct.c	20 Jan 2005 09:34:36 -0000	1.51.2.7
+++ struct.c	9 Feb 2005 15:52:57 -0000
@@ -214,2 +214,6 @@ make_struct(name, members, klass)
 	ID id = SYM2ID(RARRAY(members)->ptr[i]);
+	if (!rb_is_local_id(id)) {
+	    rb_raise(rb_eNameError, "`%s' is not proper name for a struct member",
+		     rb_id2name(id));
+	}
 	if (i<10) {