EndianStream needs to be reexamined!

Bruno Medeiros brunodomedeiros+spam at com.gmail
Tue Apr 29 05:32:10 PDT 2008


Anders F Björklund wrote:
> Bruno Medeiros wrote:
> 
>>>> Gotta say, told ya so.
>>>
>>> Isn't this what the compiler warnings do ?
>>>
>>> "warning - std/stream.d(2217): class std.stream.EndianStream Stream 
>>> read is hidden in EndianStream"
>>>
>>> --anders
>>
>> Yes, it's what they do *now*.
> 
> Glad you were vindicated in the end. Too bad the error is still there,
> the above warning is from compiling the Phobos in DMD 2.013 with -w...
> 
>> But when the feature was first introduced they threw a runtime error. 
>> I warned against it from the start:
>> "There is a problem the moment the D subclass doesn't override all
>> overloads, so that can be detected at compile time and an error issued
>> right there."
>> (http://www.digitalmars.com/webnews/newsgroups.php?art_group=digitalmars.D&article_id=56419) 
> 
> 
> Yeah, Walter has a preference for "runtime warnings" such as exceptions.
> And since warnings are optional, you can still miss them apparently...
> 
> --anders

Hum, another distraction of mine, I read the changelog and thought an 
error had been added, not a warning. I'm not sure which of the two I prefer.

-- 
Bruno Medeiros - Software Developer, MSc. in CS/E graduate
http://www.prowiki.org/wiki4d/wiki.cgi?BrunoMedeiros#D



More information about the Digitalmars-d mailing list