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

Proposal for an aux tagged field iterator API #1354

Merged
merged 2 commits into from
Aug 30, 2022
Merged

Proposal for an aux tagged field iterator API #1354

merged 2 commits into from
Aug 30, 2022

Commits on Aug 24, 2022

  1. Add bam_aux_first()/bam_aux_next() tagged aux field iterator API 

    Add new API functions for iterating through a BAM record's aux fields,
inline accessor methods for field tag and type (or code can continue
to use s-2 and *s), and a variant of bam_aux_del() that returns the
(updated) iterator to the following field (for use in iterator-based
loops that delete fields).

Add test cases for the new API functions.
    @jmarshall
    jmarshall committed Aug 24, 2022
    Configuration menu
    Copy the full SHA
    10d63e1 View commit details
    Browse the repository at this point in the history

  2. Document that bam_aux_del()'s s parameter must be non-NULL (& refor… 

    …mat)
    @jmarshall
    jmarshall committed Aug 24, 2022
    Configuration menu
    Copy the full SHA
    78ee824 View commit details
    Browse the repository at this point in the history