Skip to content

Integrate form #424

Open
Open
@burrbull

Description

@burrbull
Member

What are you think about integrate form functionality in svd2rust (optional)?

When this option is enabled we can do not merge mod TokenStreams in one (and then split with form) and write them in different files directly. I think this can give us significant speed-up in this case.

@therealprof

Activity

changed the title [-]Intergate `form`[/-] [+]Integrate `form`[/+] on Jan 7, 2020
therealprof

therealprof commented on Jan 7, 2020

@therealprof
Contributor

Yes, I was thinking about that. I was also looking into integrating with rustfmt directly since the generated code is such a hot mess; quite obviously the code generation/parsing cabilities of syn/quote/proc_macro2 were never meant to generate code stored to disk but only non-persistent code to be compiled on-the-fly.

linked a pull request that will close this issue on Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      Participants

      @burrbull@therealprof

      Issue actions

        Integrate `form` · Issue #424 · rust-embedded/svd2rust