Add option to print help message

alberthdev / spasm-ng

SPASM-ng is a z80 assembler with extra features to support development for TI calculators.

GNU General Public License v2.0

97 stars 23 forks source link

Add option to print help message #44

Closed C-Elegans closed 7 years ago

C-Elegans commented 7 years ago

Refactor the help message into its own function Add -h and -H as options to print the help message and exit

coveralls commented 7 years ago

Coverage decreased (-0.05%) to 34.481% when pulling 6bc86696576151373f9d9613904b998626ac43e9 on C-Elegans:dev into 19235dfb28f945d61d8afed7d4f95070dabfd81e on alberthdev:master.

alberthdev commented 7 years ago

Looks great, thanks for the contribution! Somewhat curious, but could you potentially fix a bug of sorts? It looks like invalid options are not being recognized, e.g.:

albert@debian:~/spasm-ng$ ./spasm -Z
No input file specified

(FYI, this isn't a bug on your side... I just noticed this in spasm-ng master due to me testing out your commit!)

If you want to do this separately from this PR, just let me know and I'll go ahead and merge this in before you do any changes. (Feel free to open an issue as well to track this problem!)

coveralls commented 7 years ago

Coverage increased (+8.8%) to 43.288% when pulling 87dad9e42e155ea95ba3ca527f58d66b7373d42a on C-Elegans:dev into 19235dfb28f945d61d8afed7d4f95070dabfd81e on alberthdev:master.

coveralls commented 7 years ago

Coverage increased (+10.2%) to 44.749% when pulling d3d34d9d523a2b462dbcad9d5ab02284f21bb266 on C-Elegans:dev into 19235dfb28f945d61d8afed7d4f95070dabfd81e on alberthdev:master.

alberthdev commented 7 years ago

Fixing post merge.