wcarhart / koi

Bashful argument parsing

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add illegal long options

wcarhart opened this issue · comments

Some words should not be allow to be long options. For example:

  • continue
  • IFS
  • etc.

There should be a check in __addarg to guarantee this does not occur.

Upon reflection, we won't have a check for this, but there is a note in the documentation about how it's not wise to use variables like IFS because it will overwrite the system/builtin ones.

Decided to reopen, will check for variable names (BASH, IFS, etc.) rather than builtins (echo) and key words (if, else).