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.

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

update to use htsFile in synced reader (for #1862) #1868

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vasudeva8
Copy link
Contributor

Fixes #1862
Added bcf_sr_add_hreader to add already opened htsfile to reader list.
The file added can be set to close along with reader or closed explicitly by user. Aux data in synced_bcf_reader.c is updated to hold the file closure status.
When the file name is passed along with file pointer, the index can be anywhere and mentioned using ##idx## method with file name. As file name in htsfile does not maintain index information, the index has to be present in default location with default name (i.e. same location as file with file name.) when no filename is passed to this method.

bcf_sr_add_reader updated to open the file and passes the file to bcf_sr_add_hreader with autoclose enabled.

Test utility and script updated to use the new interface. With -u argument, it uses the new interface where file pointer is made and used.

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.

feat: request, make synced_bcf_reader::bcf_sr_add_reader equivalent that accepts htsFile
1 participant