Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed lots of unused variables #165

Merged
merged 3 commits into from Jun 27, 2011
Merged

Conversation

mrmonday
Copy link
Contributor

This significantly cuts down on the number of warnings produced when compiling dmd with -Wall.

@WalterBright
Copy link
Member

Some of these are necessary for commented out debugging printf's. Unfortunately, that means it's not a straightforward merge.

@mrmonday
Copy link
Contributor Author

I'll scan through the commit and re-add in the ones needed for debug printfs either commented out or in #if 0 blocks. I thought I managed to get all of them as I went through, seems not though!

@mrmonday
Copy link
Contributor Author

That took forever D; Been through every diff and re-added variables that were used elsewhere. Also reverted my unintended change to posix.mak.

@WalterBright
Copy link
Member

Thanks for doing all that work!

WalterBright added a commit that referenced this pull request Jun 27, 2011
Removed lots of unused variables
@WalterBright WalterBright merged commit eccc23a into dlang:master Jun 27, 2011
@WalterBright
Copy link
Member

Had to revert much of this becase:

  1. The C++ compiler would no longer build
  2. The D compiler crashes building the library

braddr pushed a commit to braddr/dmd that referenced this pull request Sep 15, 2011
Issue 6424 - std.traits.hasElaborateAssign is limited
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants