samtools / htslib

C library for high-throughput sequencing data formats
Other
805 stars 446 forks source link

Check if VCF POS column could be fully parsed #1575

Closed pd3 closed 1 year ago

pd3 commented 1 year ago

and throw an error if not.

Resolves #1570

daviesrob commented 1 year ago

Looks OK. I notice that the error reporting for the "Position value '%s' is too large" case could be improved by using tmp as well. Would you like me to add that on merge?

pd3 commented 1 year ago

Good idea, thank you

daviesrob commented 1 year ago

Rebased with adjusted message...