Skip to content

Comments

Suppress spurious _error messages + six other bugfixes.#9

Merged
17 commits merged intodlang:masterfrom
don-clugston-sociomantic:master
Mar 1, 2011
Merged

Suppress spurious _error messages + six other bugfixes.#9
17 commits merged intodlang:masterfrom
don-clugston-sociomantic:master

Conversation

@donc
Copy link
Collaborator

@donc donc commented Feb 20, 2011

The error message fix (bug 4329) is the big one. As well as dramatically improving error message quality, it should help a bit with stability, since it means less code is exposed to nonsense syntax trees. The other bugs are pretty simple.
These commits include test cases for the test suite, and also fixes for D1. So you need to pull from donc:dmd-1.x into dmd-1.x, as well.
This will probably be my last pull request from my master branch.

4033 Error: base class is forward referenced
4135 Regression(1.034): ICE(statement.c): mixin in bad foreach, D1 only
4322 "void initializer has no value" on struct/union members initialized to "void"
4329 Do not show error messages that refer to __error
4750 fail_compilation/fail225.d causes dmd to segv
5482 Crash with align(0)
5524 [CTFE] Trouble with typesafe variadic function

Don Clugston and others added 17 commits February 7, 2011 16:20
This one is simple -- static arrays of zero length don't need an initializer.
Patch by robert@octarineparrot
Make this a parsing error, just as negative integers are.
This suppresses all spurious errors in the test cases in the bug report.
The "struct literal has no effect" error message is not spurious, even
when there is an __error in the literal, so it is not suppressed.

Additionally, the spurious "xxx is used as a type" error message which occurs
after an error involving xxx, is now suppressed.
Not reported in 4329, but I found it in bug 5288.
…zed to "void"

Patch by Graham Fawcett.
I have checked that this doesn't cause CTFE crashes.
@WalterBright
Copy link
Member

I don't see a commit for 4750?

@WalterBright
Copy link
Member

Ok, I see the 4750 stuff now.

UplinkCoder added a commit to UplinkCoder/dmd that referenced this pull request Aug 13, 2016
UplinkCoder added a commit to UplinkCoder/dmd that referenced this pull request Aug 31, 2016
UplinkCoder added a commit to UplinkCoder/dmd that referenced this pull request Sep 1, 2016
UplinkCoder added a commit to UplinkCoder/dmd that referenced this pull request Sep 4, 2016
UplinkCoder added a commit to UplinkCoder/dmd that referenced this pull request Sep 19, 2016
UplinkCoder added a commit to UplinkCoder/dmd that referenced this pull request Sep 25, 2016
tgehr added a commit to ntrel/dmd that referenced this pull request Apr 14, 2025
…ang#9)

---------

Co-authored-by: Timon Gehr <timon.gehr@gmx.ch>
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants