Skip to content

Conversation

@Liozou
Copy link
Member

@Liozou Liozou commented May 19, 2022

Fix the issue reported by @Seelengrab at #43500 (comment).
I agree with the argument and the solution proposed there, which is implemented here.

As mentioned in the comment, if this fix is accepted, it should be backported to 1.8 too.

@johnnychen94 johnnychen94 added arrays [a, r, r, a, y, s] bugfix This change fixes an existing bug backport 1.8 Change should be backported to release-1.8 labels May 19, 2022
@KristofferC KristofferC mentioned this pull request May 23, 2022
67 tasks
@Liozou
Copy link
Member Author

Liozou commented May 23, 2022

CI failures seem unrelated.
For context, what this PR does is restoring the error-throwing behavior pre-1.8 for the particular case of reinterpret(::SingletonType, ::NonSingletonArray), except that the error is a pretty ArgumentError explaining what's happening instead of a DivideError.

@KristofferC KristofferC merged commit fc52b3f into JuliaLang:master May 24, 2022
@Liozou Liozou deleted the disallowreinterpretsingletonfromnot branch May 24, 2022 16:41
KristofferC pushed a commit that referenced this pull request May 26, 2022
KristofferC added a commit that referenced this pull request May 27, 2022
@KristofferC KristofferC removed the backport 1.8 Change should be backported to release-1.8 label May 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arrays [a, r, r, a, y, s] bugfix This change fixes an existing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants