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

Introduces Bytes, Blob, and Clob wrapper types #506

Merged
merged 3 commits into from
Apr 5, 2023
Merged

Introduces Bytes, Blob, and Clob wrapper types #506

merged 3 commits into from
Apr 5, 2023

Commits on Apr 5, 2023

  1. Introduces Bytes, Blob, and Clob types 

    This PR introduces new opaque wrappers for `Vec<u8>`, which was used
throughout APIs to represent owned blobs and clobs.

`Bytes` captures the core functionality of an owned byte array, but
does not include Ion type information.

`Blob` and `Clob` are both trivial tuple struct wrappers around
`Bytes`, adding an Ion type to the data.

The `Value::Blob` and `Value::Clob` enum variants (which already
provide an Ion type) now wrap an instance of `Bytes`.

This PR fixes #500.
    Zack Slayton committed Apr 5, 2023
    Configuration menu
    Copy the full SHA
    04ffe9d View commit details
    Browse the repository at this point in the history

  2. Adds doc comments to Bytes, Blob, and Clob

    Zack Slayton committed Apr 5, 2023
    Configuration menu
    Copy the full SHA
    9d13c9b View commit details
    Browse the repository at this point in the history

  3. Fixes doc comment test

    Zack Slayton committed Apr 5, 2023
    Configuration menu
    Copy the full SHA
    5cbe5bf View commit details
    Browse the repository at this point in the history