Daniel Shahaf

Results 382 comments of Daniel Shahaf

Note to self: audit for null argument elision.

The problem is here: https://github.com/danielshahaf/zsh-syntax-highlighting/blob/e9ab802593ebbfce5cf749b1854d2cc201b3aa68/highlighters/main/main-highlighter.zsh#L471-L482 $arg starts out as `'a'` (three characters), then `_zsh_highlight_main_highlighter_expand_path` dequotes, `_zsh_highlight_main__type` returns `alias`, and `_zsh_highlight_main__resolve_alias` doesn't find an alias named `'a'` (there's only an `a`...

Filed #479 which would affect the failure mode of this.

Remaining tasks: - [ ] Resolve all review comments - [ ] Update changelog - [ ] Fix trailing whitespace - [ ] `alias a=b; : 'a'` issue above, compare...

Need precommand_options[nice]+=':' (Can't add it as a review comment) _edit_: Done in 06ae6f9

Left: - Address https://github.com/zsh-users/zsh-syntax-highlighting/pull/471#issuecomment-357249607 - alias lookup should only be done if it's either a global alias or at command word position - Address https://github.com/zsh-users/zsh-syntax-highlighting/pull/471#discussion_r161374006 - Resolve conflicts - Merge...

> The idea is that we are still in the alias even when recursing into a nested list like a command substitution, so _highlight_list is the wrong place to local...

The C code enforces that an alias be ineligible for expansion whilst it's being expanded. It's easy to see that: `alias ls=ls` works.