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

Start parsing the chunks file with serde #31

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Start parsing the chunks file with serde #31

wants to merge 3 commits into from

Commits on Sep 18, 2024

  1. Start parsing the chunks file with serde 

    This implements a hand-written parser which scans through the `chunks` file line-by-line, and parses the various headers and line records with serde.

The most complex part here is parsing the line records.
If that complexity starts to be unreasonable, a hybrid approach is also possible in which the hand-written parser is used along with the simpler serde-based `header` parsers, and still falling back to the existing parser-combinator based parser for the line records.
    @Swatinem
    Swatinem committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    f0309bb View commit details
    Browse the repository at this point in the history

  2. Use memchr-based splitting instead of an iterator/event-based inter… 

    …face
    @Swatinem
    Swatinem committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    816d632 View commit details
    Browse the repository at this point in the history

  3. get closer to the existing parser interface dealing with report builders

    @Swatinem
    Swatinem committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    bd18f58 View commit details
    Browse the repository at this point in the history