[vim/vim] More use ga_concat_len() where the length is known (PR #19038)

5 views
Skip to first unread message

John Marriott

unread,
Dec 28, 2025, 5:03:44 PM (21 hours ago) Dec 28
to vim/vim, Subscribed

This PR changes some more calls in various source files from ga_concat() to ga_concat_len() where the length is known or can be calculated.

In addition:
In os_unix.c, in function socket_server_list_sockets() use a string_T for the strings buf and path for use in ga_concat_len() and drop uneeded variable dir.
In quickfix.c:
-> in function qf_jump_print_msg() use a string_T for the string IObuff for use in ga_concat_len().
-> in function qf_range_text() use a string_T for the the string buf for use in ga_concat_len().
In register.c, simplify function execreg_line_continuation().
In terminal.c, in function read_dump_file() use a string_T for the string prev_char for use in ga_concat_len().
In tuple.c, in function tuple_join_inner() use a string_T for the string s for use in ga_concat_len(). Also, change local struct join_T to use string_T.
In vim9type.c, in functions type_name_tuple() and type_name_func() for the string arg_type for use in ga_concat_len().

Cheers
John


You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/19038

Commit Summary

  • dacffbe More use ga_concat_len() where the length is known

File Changes

(10 files)

Patch Links:


Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/19038@github.com>

John Marriott

unread,
Dec 28, 2025, 5:12:09 PM (21 hours ago) Dec 28
to vim/vim, Push

@basilisk0315 pushed 1 commit.

  • edcb4ef Fix FreeBSD compile failure


View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/19038/before/dacffbe60f8f56b5048933007592523f820fea11/after/edcb4efba44c802b885439023f6b54a87d2e8c7f@github.com>

Reply all
Reply to author
Forward
0 new messages