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

two additional remarks in documentation #14770

Closed
wants to merge 3 commits into from
Closed

Conversation

Konfekt
Copy link
Contributor

@Konfekt Konfekt commented May 14, 2024

These two points did not seem immediately clear to everyone

The {special} argument adds additional escaping of keywords
used in Vim commands. When it is not omitted and a non-zero
number or a non-empty String (|non-zero-arg|), then special
words such as "!", "%", "#" and "<cword>" (as listed in
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you keep the items instead of words please?

@chrisbra chrisbra closed this in 5faeb60 May 15, 2024
k-takata added a commit to k-takata/vim that referenced this pull request May 16, 2024
Fix the issues introduced by vim#14770.
k-takata added a commit that referenced this pull request May 16, 2024
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request May 16, 2024
runtime(doc): clarify {special} argument for shellescape()

closes: vim/vim#14770

vim/vim@5faeb60

N/A patch:
vim-patch:c0e038b59f84

Co-authored-by: Enno <Konfekt@users.noreply.github.com>
zeertzjq added a commit to neovim/neovim that referenced this pull request May 16, 2024
runtime(doc): clarify {special} argument for shellescape()

closes: vim/vim#14770

vim/vim@5faeb60

N/A patch:
vim-patch:c0e038b59f84

Co-authored-by: Enno <Konfekt@users.noreply.github.com>
icholy pushed a commit to icholy/neovim that referenced this pull request May 17, 2024
runtime(doc): clarify {special} argument for shellescape()

closes: vim/vim#14770

vim/vim@5faeb60

N/A patch:
vim-patch:c0e038b59f84

Co-authored-by: Enno <Konfekt@users.noreply.github.com>
@@ -10242,7 +10243,8 @@ tempname() *tempname()* *temp-file-name*
is different for at least 26 consecutive calls. Example: >
:let tmpfile = tempname()
:exe "redir > " .. tmpfile
< For Unix, the file will be in a private directory |tempfile|.
< For Unix, the file will be in a private directory |tempfile|
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Somehow this comment was dropped in 5faeb60
Is this clear to everyone? I got caught off-guard once and kept wondering for a while.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It didn't get dropped, I just added the missing trailing dot back and then there was no change anymore :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When looking at https://github.com/vim/vim/pull/14770/files there's also https://github.com/vim/vim/pull/14770/files#diff-ef8946d89919ad628b9ac65c49d3067e4e017b0980cac70eb25d832a59ef677dR10247 that was meant to warn against recursive deletion of everything inside the temp dir...

Copy link
Member

@chrisbra chrisbra May 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm, I might have missed to squash the third commit, sorry, let me fix it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated and further clarified here: 5cf5301

chrisbra added a commit that referenced this pull request May 18, 2024
related: #14770

Co-authored-by: Enno <Konfekt@users.noreply.github.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
huangyingw pushed a commit to huangyingw/neovim that referenced this pull request May 31, 2024
runtime(doc): clarify {special} argument for shellescape()

closes: vim/vim#14770

vim/vim@5faeb60

N/A patch:
vim-patch:c0e038b59f84

Co-authored-by: Enno <Konfekt@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants