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

Allow using Box<[u8]>/Box<str> for bytes/string fields #878

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Allow using Box<[u8]>/Box<str> for bytes/string fields #878

wants to merge 14 commits into from

Commits on Sep 18, 2023

  1. tests: add a test to keep current Config::bytes() behaviour

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    58310f8 View commit details
    Browse the repository at this point in the history

  2. tests: compile test for bytes/vec

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    3941267 View commit details
    Browse the repository at this point in the history

  3. refacto: extract bytes type setting to helper

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    9ddc72f View commit details
    Browse the repository at this point in the history

  4. feat: allow using Box<[u8]> as the bytes type

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    c27a79b View commit details
    Browse the repository at this point in the history

  5. tests: ensure Box<[u8]> is valid as bytes type

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    612442f View commit details
    Browse the repository at this point in the history

  6. refacto: introduce StringType to introduce BoxStr config later

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    3e9e9fc View commit details
    Browse the repository at this point in the history

  7. chore: change default for String to Default::default()

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    8d56397 View commit details
    Browse the repository at this point in the history

  8. codegen: generate string="string" in prost() attribute

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    354306c View commit details
    Browse the repository at this point in the history

  9. refacto: extract field_type_tag_detailed to make future evolutions … 

    …easier
    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    6ecfc9b View commit details
    Browse the repository at this point in the history

  10. fix: use field_type_tag_detailed() for oneof members too

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    c241bae View commit details
    Browse the repository at this point in the history

  11. test: string type generation

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    c56f611 View commit details
    Browse the repository at this point in the history

  12. feat: support Box<str> as the string type

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    d368719 View commit details
    Browse the repository at this point in the history

  13. tests: ensure Box<str> is a valid string type

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    8c4096b View commit details
    Browse the repository at this point in the history

  14. fix: syn MetaNameValue members changed

    @poliorcetics
    poliorcetics committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    d8f4c27 View commit details
    Browse the repository at this point in the history