Skip to content

Add bcf_sr_get_header() macro for accessing the synced readers' headers#145

Closed
droazen wants to merge 1 commit intosamtools:developfrom
broadinstitute:broad_bcf_sr_get_header
Closed

Add bcf_sr_get_header() macro for accessing the synced readers' headers#145
droazen wants to merge 1 commit intosamtools:developfrom
broadinstitute:broad_bcf_sr_get_header

Conversation

@droazen
Copy link
Copy Markdown
Contributor

@droazen droazen commented Nov 7, 2014

No description provided.

@pd3
Copy link
Copy Markdown
Member

pd3 commented Nov 10, 2014

Thank you for this. I removed the condition, the macro should return the header regardless of the number of records in the buffer. I hope that's OK for your usage? ee7343e

@droazen
Copy link
Copy Markdown
Contributor Author

droazen commented Nov 10, 2014

Yes, your version is better -- thanks for patching our patch :) Will close this.

@droazen droazen closed this Nov 10, 2014
@droazen droazen deleted the broad_bcf_sr_get_header branch November 10, 2014 15:56
pd3 added a commit to pd3/htslib that referenced this pull request Sep 25, 2015
jkbonfield added a commit to jkbonfield/htslib that referenced this pull request Mar 11, 2026
Changelog from htscodecs:

- Prevent use of unaligned 16-bit integers from memmove.
  (PR samtools#145 Robert Davies)

- Add missing support for X_CAT in arithmetic codec.
  (PR samtools#146; Issue samtools#144 Michael Macias)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants