[Issue 952] New: Strange "Error:" prefix on some warning messages

[d-bugmail at puremagic.com]

http://d.puremagic.com/issues/show_bug.cgi?id=952

           Summary: Strange "Error:" prefix on some warning messages
           Product: D
           Version: 1.005
          Platform: PC
        OS/Version: Windows
            Status: NEW
          Keywords: diagnostic
          Severity: trivial
          Priority: P2
         Component: DMD
        AssignedTo: bugzilla at digitalmars.com
        ReportedBy: smjg at iname.com


Some of the warning messages have this strange wording error (no pun intended)
in them.  For example:

----------
int main() {
        return 1;
        return 0;
}
----------
warning - warning_unreachable.d(3): Error: statement is not reachable
----------

Why the label "Error:" here?  It isn't an error, it's a warning.  Presumably
the intention was for it to output

warning - warning_unreachable.d(3): statement is not reachable

We should get rid of the string "Error:" label especially from warning
messages.  Really, it shouldn't be in any complier messages, perhaps apart from
those for which it doesn't make sense to give a filename and line number.

You could debate whether it should be this or

warning_unreachable.d(3): Warning: statement is not reachable

but that's an aside.


--