Feature #11749: [PATCH] variable.c: remove spurious #define for globals

• Status: Closed
• Priority: Normal
• Assignee: -
• Target version: -

[normalperson (Eric Wong)]

variable.c: remove spurious #define for globals

Removing the indirection helps me with readability, at
least. It doesn't seem like there are many other places
in the Ruby code where macros are used like this.

@nobu (Nobuyoshi Nakada): They appeared in r19781 when the *_{getter,setter,marker}
methods were made public, presumably to minimize the size of the
diff.

Will commit in a week or so unless there are objections.

[nobu (Nobuyoshi Nakada)]

Eric Wong wrote:

@nobu (Nobuyoshi Nakada): They appeared in r19781 when the *_{getter,setter,marker}
methods were made public, presumably to minimize the size of the
diff.

Will commit in a week or so unless there are objections.

Yes, please fix it.

• Status changed from Open to Closed

Applied in changeset r52790.

---

variable.c: remove spurious #define for globals

Removing the indirection helps me with readability, at
least. It doesn't seem like there are many other places
in the Ruby code where macros are used like this.

[ruby-core:71735] [Feature #11749]