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

Remove <mbe::TokenTree as Clone> #95928

Merged
merged 5 commits into from
Apr 14, 2022
Merged

Remove <mbe::TokenTree as Clone> #95928

merged 5 commits into from
Apr 14, 2022

Commits on Apr 13, 2022

  1. SequenceRepetition and Delimited don't need to be Clone.

    @nnethercote
    nnethercote committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    769e2ed View commit details
    Browse the repository at this point in the history

  2. Avoid use of Lrc in mbe::Frame. 

    This is a nice performance win on some crates.
    @nnethercote
    nnethercote committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    1a70064 View commit details
    Browse the repository at this point in the history

  3. Pass a slice instead of a Vec to transcribe. 

    It avoids some unnecessary allocations.
    @nnethercote
    nnethercote committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    2657d8f View commit details
    Browse the repository at this point in the history

  4. Introduce TtHandle and use it in TokenSet. 

    This removes the last use of `<mbe::TokenTree as Clone>`. It also
removes two trivial methods on `Delimited`.
    @nnethercote
    nnethercote committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    75fd391 View commit details
    Browse the repository at this point in the history

  5. mbe::TokenTree: remove Lrc around Delimited and `SequenceRepeti… 

    …tion`.
    @nnethercote
    nnethercote committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    dd9028a View commit details
    Browse the repository at this point in the history