diff options
author | Nicolas Hillegeer <nicolas@hillegeer.com> | 2014-04-28 21:07:03 +0200 |
---|---|---|
committer | Thiago de Arruda <tpadilha84@gmail.com> | 2014-04-29 09:29:10 -0300 |
commit | eea98c4d3b695b9ca95db7ac33231e62cdf88ceb (patch) | |
tree | c48634c80de748e8260548db05a3606dddcd2770 /src/garray.h | |
parent | 767f0fae0b2f981b4563c5b5e2ffdd2b8ed8e27b (diff) | |
download | rneovim-eea98c4d3b695b9ca95db7ac33231e62cdf88ceb.tar.gz rneovim-eea98c4d3b695b9ca95db7ac33231e62cdf88ceb.tar.bz2 rneovim-eea98c4d3b695b9ca95db7ac33231e62cdf88ceb.zip |
garray: refactor and optimize ga_concat
I've checked all uses of ga_concat in the codebase:
- ex_cmds2.c
- ex_getln.c
- eval.c
- message.c
- regexp-nfa.c
- term.c
- spell.c
None of them concats the garray with itself. This makes it safe to use the
faster memcpy() instead of memmove(). This change was also documented.
Diffstat (limited to 'src/garray.h')
-rw-r--r-- | src/garray.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/garray.h b/src/garray.h index 2c7bc2f375..2a0deacef7 100644 --- a/src/garray.h +++ b/src/garray.h @@ -24,7 +24,7 @@ char_u *ga_concat_strings_sep(const garray_T *gap, const char *sep) FUNC_ATTR_NONNULL_RET; char_u *ga_concat_strings(const garray_T *gap) FUNC_ATTR_NONNULL_RET; void ga_remove_duplicate_strings(garray_T *gap); -void ga_concat(garray_T *gap, char_u *s); +void ga_concat(garray_T *gap, const char_u *restrict s); void ga_append(garray_T *gap, char c); void append_ga_line(garray_T *gap); |