"I told you so": noreturn sucks a leech and has virtually no utility

MoonlightSentinel moonlightsentinel at disroot.org
Fri Oct 15 18:32:47 UTC 2021


On Friday, 15 October 2021 at 17:13:29 UTC, Andrei Alexandrescu 
wrote:
> It has been predicted by several folks that noreturn has very 
> limited utility and a variety of weird corner cases that will 
> cause a hecatomb of complications in the language, the standard 
> library, and the implementation.

This is a claim that has yet to be seen. Most of the 
complications stem from the fact that the introduction of 
`noreturn` allows DMD to detect unreachable code more accurately. 
This caused several `statement not reachable` warnings in Phobos 
(and other libraries) because templated functions are now 
inferred as `noreturn` instead of `void`.

But that problem is not specific to `noreturn` and instead a 
general issue for templated code.

> (For starters: how the hell is "will not return" a type that's 
> supposed to do all or at least most things that types do, when 
> even the phrase defining it has no subject? It's about a 
> FUNCTION that won't return, so it should be an attribute 
> applicable to a FUNCTION!)

The name is misleading, `noreturn` represents the bottom type in 
general. But arguing about `noreturn` vs. `Nothing` vs. `<your 
choice here>` is just bikeshedding.

> In spite of these voices of reason, noreturn proceeded with a 
> definition and implementation as expected: execrable.
> 
> [...]
>
> Ten opened bugs.

The DIP is actually rather precise in it's definition and 
reasoning. The implementation included in the current release is 
incomplete and in a pretty bad state. Walter started implementing 
some very specific basics (e.g. `noreturn exit()`) and left of 
without adding tests for the examples / requirements specified by 
the DIP (many of them are rejected or crash dmd).

This is especially problematic because `noreturn` was introduced 
without a `-preview` flag. But at least the next release should 
be a lot better, I've already fixed several of the aforementioned 
issues and proposed an implementation for the missing aspects of 
that DIP.

> I was looking today through the wretched isAutodecodableString 
> (how did we ever think /that/ was a good idea?) and found a 
> number of weird occurrences of noreturn. Where? Of course, 
> where you least expect or want it.

That one is indeed nasty but mostly stems from the fact that 
`isAutodecodableString` accepts any type convertible to 
`[d]char[]` (except static arrays for some reason). That traits 
is a decent example of an overly general implementation.

> This is literally why we can't have nice things. If all that 
> investment went to literally any other proposal, it would be a 
> net positive.

This exaggeration is wrong and definitely uncalled for



More information about the Digitalmars-d mailing list