search

arshajii / ema

Fast & accurate alignment of barcoded short-reads
http://ema.csail.mit.edu
MIT License
32 stars 7 forks source link

raise MAX_CHROM_NAME_LEN #30

Closed rchikhi closed 5 years ago

rchikhi commented 5 years ago

Any reason why MAX_CHROM_NAME_LEN is set to 64? It's so low that it causes a segfault on some of my long-headered FASTA files. increasing this constant fixed the problem.

arshajii commented 5 years ago

No particular reason; at some point we should probably remove the max length and just dynamically allocate those chromosome name strings individually (unless the FASTA spec puts some limit on this, but I don't believe it does). I'll merge this for now though -- thank you!