Bug 1480 – std.stream throws the new override warning all over the place
Status
RESOLVED
Resolution
FIXED
Severity
critical
Priority
P2
Component
phobos
Product
D
Version
D2
Platform
x86
OS
Windows
Creation time
2007-09-06T20:50:00Z
Last change time
2015-06-09T01:14:16Z
Assigned to
bugzilla
Creator
spam
Comments
Comment #0 by spam — 2007-09-06T20:50:22Z
it is as simple as that, just import the std.stream module and compile with warnings enabled then the new override warning is thrown hundreds of times:
[CODE]
import std.stream;
void main()
{}
[/CODE]
this is very annoying as i always have warnings turned on.
Comment #1 by ddparnell — 2007-10-16T21:48:37Z
Messages like ...
phobos\std\stream.d(1942): function std.stream.File.size overrides base class function std.stream.Stream.size, but is not marked with 'override'
This is a major showstopper for me too as I always use the -w switch.
Comment #2 by braddr — 2007-10-17T00:17:28Z
Whoops, missed that one as it's win32 only (and most of my development is on linux). The fix is checked in to the 2.x branch for the next release.
I dunno that I'd call it a show stopper, since it's trivial to edit the .d file, or stop compiling with warnings. Show stoppers are generally reserved for problems that can't be worked around. Either way, the fix is lined up now.
Comment #3 by spam — 2007-11-02T17:14:25Z
did i miss something ? it is not fixed in the 2.007 release is it ?
Comment #4 by braddr — 2007-11-03T16:30:22Z
It's fixed, just missing from the change log (mail sent to walter to get that fixed already). If there's an override annotation that's missing still, shout.
Comment #5 by spam — 2007-11-03T21:14:43Z
i am sorry, my bad, as far as i can see it is solved in dmd2.007