Skip to content

The null case of a nullable-poiner enum might not be nullary. - #6134

Closed
jld wants to merge 1 commit into
rust-lang:incomingfrom
jld:issue-6117
Closed

The null case of a nullable-poiner enum might not be nullary.#6134
jld wants to merge 1 commit into
rust-lang:incomingfrom
jld:issue-6117

Conversation

@jld

@jldjld commented Apr 30, 2013

Copy link
Copy Markdown
Contributor

Cases like Either<@int,()> have a null case with at most one value but
a nonzero number of fields; if we misreport this, then bad things can
happen inside of, for example, pattern matching.

Closes#6117.

Cases like `Either<@int,()>` have a null case with at most one value but
a nonzero number of fields; if we misreport this, then bad things can
happen inside of, for example, pattern matching.
Closesrust-lang#6117.
bors added a commit that referenced this pull request Apr 30, 2013
Cases like `Either<@int,()>` have a null case with at most one value but
a nonzero number of fields; if we misreport this, then bad things can
happen inside of, for example, pattern matching.
Closes#6117.
@borsbors closed this Apr 30, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Nov 20, 2020
Check when `from_utf8` is called from sliced byte array from string
---
*Please keep the line below*
changelog: Fixrust-lang#5487: Add linter to check when `from_utf8` is called from sliced byte array from string.
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
 * Chalk very recently (like an hour ago) merged a fix that prevents rust analyzer from panicking. This allows it to be usable again for code that hits those situations. See rust-lang#6134, rust-lang#6145, Probably rust-lang#6120
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
6161: Bump chalk to use latest git to get upstream fix r=jonas-schievink a=Ameobea
* Chalk very recently (like an hour ago) merged a fix that prevents rust analyzer from panicking. This allows it to be usable again for code that hits those situations. See rust-lang#6134, rust-lang#6145, Probably rust-lang#6120
Co-authored-by: Casey Primozic <me@ameo.link>
Sign up for freeto 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.

3 participants

@jld@catamorphism@bors