Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

rustc_transmute cleanups #116466

Merged
merged 5 commits into from
Oct 6, 2023
Merged

rustc_transmute cleanups #116466

merged 5 commits into from
Oct 6, 2023

Commits on Oct 5, 2023

  1. Remove an unnecessary pub(crate).

    @nnethercote
    nnethercote committed Oct 5, 2023
    Configuration menu
    Copy the full SHA
    80bf688 View commit details
    Browse the repository at this point in the history

  2. Make the comment order match variant declaration order.

    @nnethercote
    nnethercote committed Oct 5, 2023
    Configuration menu
    Copy the full SHA
    4091936 View commit details
    Browse the repository at this point in the history

Commits on Oct 6, 2023

  1. Remove map_layouts. 

    As per the `FIXME` comment, it's an abstraction that makes the code
harder to read.
    @nnethercote
    nnethercote committed Oct 6, 2023
    Configuration menu
    Copy the full SHA
    449b84c View commit details
    Browse the repository at this point in the history

  2. Fix a comment. 

    It was duplicated from the method above.
    @nnethercote
    nnethercote committed Oct 6, 2023
    Configuration menu
    Copy the full SHA
    73420fc View commit details
    Browse the repository at this point in the history

  3. Remove the MaybeTransmutableQuery<&'l Dfa<...>, C> impl. 

    Because there is also a `MaybeTransmutableQuery<Dfa<...>, C>` impl, and
we don't need both.
    @nnethercote
    nnethercote committed Oct 6, 2023
    Configuration menu
    Copy the full SHA
    29ed8e4 View commit details
    Browse the repository at this point in the history