Feature #5157: [PATCH] add -Wunused-variable to CFLAGS

• Status: Closed
• Assignee: kosaki (Motohiro KOSAKI)
• Target version: 2.0.0

[normalperson (Eric Wong)]

Unused variables are ugly.

As an often-careless C extension maintainer, I end up with a lot of unused
variables, so this compiler flag will propagate to extensions I maintain.

The core of Ruby is remarkably free of unused variables despite the
current lack of -Wunused-variable, however I've also attached a trivial
patch to the openssl extension that removes a few unused variables.

[kosaki (Motohiro KOSAKI)]

Eric, I think you contributed a lot of patches and you've proved you have enough development and communication skill. So, I'd like to suggest you get a commit right and commit it by yourself.
I'll talk about it with matz and shugo.

[normalperson (Eric Wong)]

Motohiro KOSAKI [e-mail] wrote:

Eric, I think you contributed a lot of patches and you've proved you
have enough development and communication skill. So, I'd like to
suggest you get a commit right and commit it by yourself.
I'll talk about it with matz and shugo.

Thanks, but it's not my personal belief/style to accept commit rights.
I don't like the idea of shared commit, at all, actually (I prefer the
BDFL model) and I have trouble accepting responsibility :)

--
Eric Wong

[kosaki (Motohiro KOSAKI)]

• Status changed from Open to Assigned
• Assignee set to kosaki (Motohiro KOSAKI)

I've assigned it to me then.

[kosaki (Motohiro KOSAKI)]

• Status changed from Assigned to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r32883.
Eric, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• configure.in: add -Wunused-variable to default CFLAGS.
  Patch by Eric Wong. [Feature #5157] [ruby-core:38798]