Project

General

Profile

Feature #5157

[PATCH] add -Wunused-variable to CFLAGS

Added by normalperson (Eric Wong) about 9 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Normal
Target version:
[ruby-core:38798]

Description

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.


Files

Updated by kosaki (Motohiro KOSAKI) about 9 years ago

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.

Updated by normalperson (Eric Wong) about 9 years ago

Motohiro KOSAKI kosaki.motohiro@gmail.com 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

Updated by kosaki (Motohiro KOSAKI) about 9 years ago

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

I've assigned it to me then.

#4

Updated by kosaki (Motohiro KOSAKI) about 9 years ago

  • 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]

Also available in: Atom PDF