Skip to content

Make QueryIdx public #262

Closed
Closed
Make QueryIdx public#262

Description

@TheRustifyer
TheRustifyer
opened on Nov 22, 2022

Hello. Following this issue a lot of people are interested in make the QueryIdx trait public.

By make it public, you will allow to your consumers to write generic code that can abstract the get() or try_get() implementations far beyond the actual implementations, that is monomorphize with some &str or usize those methods over Row.

Would be possible to make a public reexport of the QueryIdx trait?

Activity

  1. pimeys commented on Nov 26, 2022

    @pimeys
    pimeys
    on Nov 26, 2022
    Contributor

    Sure, it should be an easy PR to do. Do you want to try?

  2. TheRustifyer commented on Jan 9, 2023

    @TheRustifyer
    TheRustifyer
    on Jan 9, 2023
    Author

    Sure, it should be an easy PR to do. Do you want to try?

    Oh, I completly forgot about this one. Isn't this change made yet?

  3. MattJackson commented on Sep 18, 2026

    @MattJackson
    MattJackson
    on Sep 18, 2026
    Contributor

    FYI: this is implemented in #442 (s4), currently in review. It'll be closed when that PR merges.

  4. MattJackson commented on Sep 24, 2026

    @MattJackson
    MattJackson
    on Sep 24, 2026
    Contributor

    Fixed in #442, shipped in v0.13.0 (https://github.com/tiberius-rs/tiberius/releases/tag/v0.13.0). Closing.

  5. MattJackson
    closed this as completedon Sep 24, 2026
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

      No branches or pull requests

      Issue actions