dnbaker/dashing2

Specifying `--verbose` tends to give segmentation fault

BenLangmead opened this issue · 2 comments

$ dashing2 sketch --verbose
#Calling Dashing2 version v2.1.11-1-g3b71 with command '/home/blangme2/scr16_blangme2/langmead/dashing2/dashing2-64 /home/blangme2/scr16_blangme2/langmead/dashing2/dashing2-64 sketch --verbose'
Segmentation fault (core dumped)
$ dashing2 cmp --verbose
#Calling Dashing2 version v2.1.11-1-g3b71 with command '/home/blangme2/scr16_blangme2/langmead/dashing2/dashing2-64 /home/blangme2/scr16_blangme2/langmead/dashing2/dashing2-64 cmp --verbose'
Segmentation fault (core dumped)

I don't think there's a verbose option currently. We could add it in (maybe change the VERBOSE_ONLY macro to be controlled by a runtime parameter instead of a macro) or add in some logging and control it via a flag.

This should be fixed now. (The --verbose option doesn't do anything currently, but there's no segfault.) Similarly, there is now a check for unexpected CLI arguments, so an error will be thrown instead of SEGV.