Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

STRALGO LCS sytanx spec not accurate #235

Closed
patpatbear opened this issue Mar 8, 2021 · 0 comments
Closed

STRALGO LCS sytanx spec not accurate #235

patpatbear opened this issue Mar 8, 2021 · 0 comments

Comments

@patpatbear
Copy link

patpatbear commented Mar 8, 2021

currently STRALGO LCS syntax spec is:

STRALGO LCS [KEYS ...] [STRINGS ...] [LEN] [IDX] [MINMATCHLEN <len>] [WITHMATCHLEN]

which IMHO is not accurate, and might be confusing:

  • KEYS ..., STRINGS ... seems like the LCS algo can accept arbitary number of KEY or STRING
  • [KEYS ...] [STRINGS ...] seems like LCS can accept either KEYS, STRINGS , or both or neither

I suggest the one used in the source code comment:

STRALGO LCS [IDX] [MINMATCHLEN <len>] [WITHMATCHLEN]  STRINGS <string> <string> | KEYS <keya> <keyb> 
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant