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

support simple neighbors #1

Closed
kiedtl opened this issue Jan 21, 2020 · 1 comment
Closed

support simple neighbors #1

kiedtl opened this issue Jan 21, 2020 · 1 comment

Comments

@kiedtl
Copy link

kiedtl commented Jan 21, 2020

In the README, you state that you don't support simple neighbors. I'm curious to know why.

@nullgemm
Copy link
Owner

Because I think they are misleading :D

With test -abc42 1337 23 what is assigned to a, b and c? I would say c gets 42, a gets 1337 and b 23, but that would mean the first number is assigned to the last option, and then everything is processed in the right order again, weird...
If it is not the case and a gets 42, b 1337 and c 23, then why removing a simple ? It makes it look like the value is connected to the c when in reality it is not the case.
We could also implement them on non-compound options only, but saving a or a = isn't really worth the extra code IMHO.
I also happen to not have a perfect vision at all, and separating the options and their values helps improving readability.

So for all these reasons I chose not to add simple neighbors.

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

2 participants