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

slices: update Int.Scan() to parse empty slices #791

Merged
merged 1 commit into from Nov 1, 2022
Merged

slices: update Int.Scan() to parse empty slices #791

merged 1 commit into from Nov 1, 2022

Commits on Nov 1, 2022

  1. slices: update Int.Scan() to parse empty slices 

    Previously, Int.Scan() returned the slice '[0]' when reading an empty
slice of ints from the database. This is a bug, as the output should
also be empty. This patch fixes the bug and adds regression tests.
Additionally, the code is updated to percolate up the errors from
integer to string conversion, which was previously swallowed.
    @carloruiz @sio4
    carloruiz authored and sio4 committed Nov 1, 2022
    Copy the full SHA
    e33e70c View commit details
    Browse the repository at this point in the history