diff options
author | zeertzjq <zeertzjq@outlook.com> | 2024-04-20 19:31:00 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-04-20 19:31:00 +0800 |
commit | 0ea38c9a53dfcff17703ea22f701ed1cc5bbd7d3 (patch) | |
tree | 60fa7289ee8fc164da54b905d030a09671297867 /src/nvim/syntax.c | |
parent | 4d52b0cf670502caf81b70f2f1e6f8c548b78f58 (diff) | |
download | rneovim-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/nvim/syntax.c')
-rw-r--r-- | src/nvim/syntax.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/nvim/syntax.c b/src/nvim/syntax.c index 76824879d7..b63d2a729d 100644 --- a/src/nvim/syntax.c +++ b/src/nvim/syntax.c @@ -2733,7 +2733,7 @@ static int check_keyword_id(char *const line, const int startcol, int *const end // Must make a copy of the keyword, so we can add a NUL and make it // lowercase. char keyword[MAXKEYWLEN + 1]; // assume max. keyword len is 80 - xstrlcpy(keyword, kwp, (size_t)kwlen + 1); + xmemcpyz(keyword, kwp, (size_t)kwlen); keyentry_T *kp = NULL; @@ -4949,7 +4949,7 @@ static int get_id_list(char **const arg, const int keylen, int16_t **const list, do { for (end = p; *end && !ascii_iswhite(*end) && *end != ','; end++) {} char *const name = xmalloc((size_t)(end - p) + 3); // leave room for "^$" - xstrlcpy(name + 1, p, (size_t)(end - p) + 1); + xmemcpyz(name + 1, p, (size_t)(end - p)); if (strcmp(name + 1, "ALLBUT") == 0 || strcmp(name + 1, "ALL") == 0 || strcmp(name + 1, "TOP") == 0 |