Daniel Shahaf

Results 356 comments of Daniel Shahaf

Just noticed that `_perforce` lists pws as maintainer (line 4). He doesn't seem to have a github account,¹ but we can get in touch with him via -workers@. So, @kting28,...

> However, I had to introduce a dependency upon `grep` to exclude user-defined options (which are prefixed with an `@` symbol), which seemed conceptually cleaner than updating `find_new`, `find_old`, and...

Feel free to post this to [email protected]; you may get a faster response there. There's no need to include ChangeLog hunks in patches (but thanks for doing so).

Could you add self-contained reproduction instructions? For instance, can one just bind-mount a path with spaces in the basename? If it's reproducible even without mount(8), even better :)

Thanks; I can reproduce the bug using the bind-mount instructions you added in your edit. :+1:

Just wanted to add that keeping the original code but changing `matches+=( ${…} )` to `matches+=( ${(q)${…}} )` also fixes the problem. I'm not sure which change is better (if...

This isn't specific to viins mode; you can get it in vicmd mode if you use a bar cursor there too. AFAICT z-sy-h does instruct the terminal to render both...

> AFAICT z-sy-h does instruct the terminal to render both parentheses in > standout, but the terminal doesn't do so for the parenthesis at the > cursor position. So, set...

> Yes, you are correct, seems like the bar cursor on its own is enough to confuse it. More or less. The `cursor-matchingbracket` style does not cover the bracket under...

> Thank you for triaging/solving them! You're welcome, and sorry for the confusion above. > Btw, I just noticed that the `change_cursor` function I wrote in the first comment has...