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

highlight nice foo #168

Closed
anntzer opened this issue Apr 27, 2015 · 2 comments
Closed

highlight nice foo #168

anntzer opened this issue Apr 27, 2015 · 2 comments
Milestone

Comments

@anntzer
Copy link
Contributor

anntzer commented Apr 27, 2015

Similarly to #107, which highlights the command after sudo, one could highlight the command after nice.

@danielshahaf
Copy link
Member

The problem is options parsing: see 19981ef and 3b3a58b for an analogous example.

There are many other commands that accept an execve-able argument list; some common ones are env, xargs, and find ... -exec. Perhaps it would be better, instead of hardcoding N separate option parsers, to hook into completion system (issue #10) and ask compsys to identify command words for us?

compsys already solved the "parse find's command line" problem, and I don't want to have to repeat that exercise. ☺

[ That said... if it's possible to support nice, or env, or any other single utility, in a simple manner without adding lots of twisty codepaths, that might be worthwhile as an interim measure until #10 is fixed ]

This was referenced Dec 4, 2015
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Jul 28, 2016
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Jul 28, 2016
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Jul 28, 2016
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Jul 28, 2016
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Jul 28, 2016
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Aug 16, 2016
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Aug 20, 2016
@anntzer
Copy link
Contributor Author

anntzer commented Aug 28, 2016

Let's add strace to the list of affected commands, too.

danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Dec 25, 2017
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Dec 25, 2017
danielshahaf added a commit to danielshahaf/zsh-syntax-highlighting that referenced this issue Dec 25, 2017
phy1729 pushed a commit to phy1729/zsh-syntax-highlighting that referenced this issue Mar 26, 2018
phy1729 pushed a commit to phy1729/zsh-syntax-highlighting that referenced this issue Sep 30, 2018
phy1729 pushed a commit to phy1729/zsh-syntax-highlighting that referenced this issue Oct 11, 2018
@phy1729 phy1729 added this to the 0.7.0 milestone Oct 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants