aboutsummaryrefslogtreecommitdiff
path: root/src/clint.py
diff options
context:
space:
mode:
authorzeertzjq <zeertzjq@outlook.com>2024-04-20 19:31:00 +0800
committerGitHub <noreply@github.com>2024-04-20 19:31:00 +0800
commit0ea38c9a53dfcff17703ea22f701ed1cc5bbd7d3 (patch)
tree60fa7289ee8fc164da54b905d030a09671297867 /src/clint.py
parent4d52b0cf670502caf81b70f2f1e6f8c548b78f58 (diff)
downloadrneovim-0ea38c9a53dfcff17703ea22f701ed1cc5bbd7d3.tar.gz
rneovim-0ea38c9a53dfcff17703ea22f701ed1cc5bbd7d3.tar.bz2
rneovim-0ea38c9a53dfcff17703ea22f701ed1cc5bbd7d3.zip
refactor: add xmemcpyz() and use it in place of some xstrlcpy() (#28422)
Problem: Using xstrlcpy() when the exact length of the string to be copied is known is not ideal because it requires adding 1 to the length and an unnecessary strlen(). Solution: Add xmemcpyz() and use it in place of such xstrlcpy() calls.
Diffstat (limited to 'src/clint.py')
-rwxr-xr-xsrc/clint.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/clint.py b/src/clint.py
index 9bca634171..41058469b1 100755
--- a/src/clint.py
+++ b/src/clint.py
@@ -1989,12 +1989,12 @@ def CheckLanguage(filename, clean_lines, linenum, error):
match = Search(r'\b(strncpy|STRNCPY)\b', line)
if match:
error(filename, linenum, 'runtime/printf', 4,
- 'Use xstrlcpy or snprintf instead of %s (unless this is from Vim)'
+ 'Use xstrlcpy, xmemcpyz or snprintf instead of %s (unless this is from Vim)'
% match.group(1))
match = Search(r'\b(strcpy)\b', line)
if match:
error(filename, linenum, 'runtime/printf', 4,
- 'Use xstrlcpy or snprintf instead of %s' % match.group(1))
+ 'Use xstrlcpy, xmemcpyz or snprintf instead of %s' % match.group(1))
match = Search(r'\b(STRNCAT|strncat|strcat|vim_strcat)\b', line)
if match:
error(filename, linenum, 'runtime/printf', 4,