'main': Highlight backslash escape sequences outside quotes.#706
'main': Highlight backslash escape sequences outside quotes.#706danielshahaf wants to merge 1 commit intozsh-users:masterfrom
Conversation
Fixes zsh-users#631. WIP: test expectations should be updated (12 failures including cthulhu)
ad8358c to
c8f5d10
Compare
|
@phy1729 Sanity check please before I update test expectations? «make quiet-test» output |
|
path_prefix3 I'm almost regretting Cthulhu exists now, but I'm glad it verified it works when nested in side backticks with |
|
Thanks for the review.
I'll have to verify cthulhu manually, I guess… |
And if that's not it, I'll next check whether it's a bug in this PR or a side-effect of #705 that's exposed by this PR. |
|
I checked the diff for cthulhu. I was just noting that I'm glad the triple backslash-un-quoted-argument was highlighted in entirety as it should be. From a quick test typing the PREBUFFER and BUFFER of path_prefix3 and dumping PREBUFFER and BUFFER, I concur the test is wrong and is missing a |
|
I checked the diff for cthulhu.
Thanks!
I was just noting that I'm glad the
triple backslash-un-quoted-argument was highlighted in entirety as it
should be.
*nod* Understood.
From a quick test typing the PREBUFFER and BUFFER of path_prefix3 and
dumping PREBUFFER and BUFFER, I concur the test is wrong and is missing
a `$'\n'` at the end of PREBUFFER.
Will fix.
|
…revent this from recurring. Discussed: #706 (comment)
Done in f6471db. |
|
Stray space after |
Fixes #631.
WIP: test expectations should be updated (12 failures including cthulhu)