Skip to content

Conversation

@ntrel
Copy link
Contributor

@ntrel ntrel commented Jul 20, 2025

Fixes #4264.

Fix backwards noreturn implicit conversion item (oops).
Document qualified array/pointer element type needing qualified void element type.
Add example.
Add link to qualifier conversions.

Fix backwards `noreturn` implicit conversion item (oops).
Document qualified array/pointer element type needing qualified void element
type.
Add example.
Add link to qualifier conversions.
@dlang-bot
Copy link
Contributor

Thanks for your pull request and interest in making D better, @ntrel! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please verify that your PR follows this checklist:

  • My PR is fully covered with tests (you can see the coverage diff by visiting the details link of the codecov check)
  • My PR is as minimal as possible (smaller, focused PRs are easier to review than big ones)
  • I have provided a detailed rationale explaining my changes
  • New or modified functions have Ddoc comments (with Params: and Returns:)

Please see CONTRIBUTING.md for more information.


If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment.

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Copy link
Contributor

@dkorpel dkorpel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix backwards noreturn implicit conversion item (oops).

I always have to stop and think about that, it's one of those things to very easily get backwards :)

---
void main()
{
noreturn n;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is going to raise some eyebrows, but the spec is there to describe semantics, not idiomatic code. I'm considering adding a bit of a rationale section at some point.

@dkorpel dkorpel merged commit fc2be54 into dlang:master Jul 23, 2025
2 checks passed
@ntrel ntrel deleted the impl-conv branch July 23, 2025 11:06
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.

Spec for implicit conversions is wrong for noreturn and incomplete for void array/pointer

3 participants