diff options
author | Luuk van Baal <luukvbaal@gmail.com> | 2023-02-04 10:43:20 +0100 |
---|---|---|
committer | Luuk van Baal <luukvbaal@gmail.com> | 2023-02-04 10:43:20 +0100 |
commit | 60ab22dfa6c9888fe8e261f185bfe4b0ab72956e (patch) | |
tree | 3d9d156347e7af58cc00dd8c2af240e983a1042b | |
parent | 7a710f5f0840724b7352ab36cfa9b8db5d533657 (diff) | |
download | rneovim-60ab22dfa6c9888fe8e261f185bfe4b0ab72956e.tar.gz rneovim-60ab22dfa6c9888fe8e261f185bfe4b0ab72956e.tar.bz2 rneovim-60ab22dfa6c9888fe8e261f185bfe4b0ab72956e.zip |
refactor(column): remove unused build_statuscol_str() arguments
Problem: `build_statuscol_str()` still has arguments that were
necessary for building a status column string in
`number_width()`, which was abandoned in #22094.
Solution: Remove unused arguments.
-rw-r--r-- | src/nvim/drawline.c | 6 | ||||
-rw-r--r-- | src/nvim/statusline.c | 11 |
2 files changed, 6 insertions, 11 deletions
diff --git a/src/nvim/drawline.c b/src/nvim/drawline.c index 01ff207c2b..7a64f8ac9d 100644 --- a/src/nvim/drawline.c +++ b/src/nvim/drawline.c @@ -442,14 +442,12 @@ static void get_statuscol_str(win_T *wp, linenr_T lnum, int row, int startrow, i if (wp->w_statuscol_line_count != wp->w_nrwidth_line_count) { wp->w_statuscol_line_count = wp->w_nrwidth_line_count; set_vim_var_nr(VV_VIRTNUM, 0); - build_statuscol_str(wp, wp->w_nrwidth_line_count, 0, stcp->width, - ' ', stcp->text, &stcp->hlrec, stcp); + build_statuscol_str(wp, wp->w_nrwidth_line_count, 0, stcp); stcp->width += stcp->truncate; } set_vim_var_nr(VV_VIRTNUM, virtnum); - int width = build_statuscol_str(wp, lnum, relnum, stcp->width, - ' ', stcp->text, &stcp->hlrec, stcp); + int width = build_statuscol_str(wp, lnum, relnum, stcp); // Force a redraw in case of error or when truncated if (*wp->w_p_stc == NUL || (stcp->truncate > 0 && wp->w_nrwidth < MAX_NUMBERWIDTH)) { if (stcp->truncate) { // Avoid truncating 'statuscolumn' diff --git a/src/nvim/statusline.c b/src/nvim/statusline.c index 6ad1f31143..a1523f6574 100644 --- a/src/nvim/statusline.c +++ b/src/nvim/statusline.c @@ -885,11 +885,8 @@ void draw_tabline(void) /// Build the 'statuscolumn' string for line "lnum". When "relnum" == -1, /// the v:lnum and v:relnum variables don't have to be updated. /// -/// @param hlrec HL attributes (can be NULL) -/// @param stcp Status column attributes (can be NULL) /// @return The width of the built status column string for line "lnum" -int build_statuscol_str(win_T *wp, linenr_T lnum, long relnum, int maxwidth, int fillchar, - char *buf, stl_hlrec_t **hlrec, statuscol_T *stcp) +int build_statuscol_str(win_T *wp, linenr_T lnum, long relnum, statuscol_T *stcp) { bool fillclick = relnum >= 0 && lnum == wp->w_topline; @@ -900,8 +897,8 @@ int build_statuscol_str(win_T *wp, linenr_T lnum, long relnum, int maxwidth, int StlClickRecord *clickrec; char *stc = xstrdup(wp->w_p_stc); - int width = build_stl_str_hl(wp, buf, MAXPATHL, stc, "statuscolumn", OPT_LOCAL, fillchar, - maxwidth, hlrec, fillclick ? &clickrec : NULL, stcp); + int width = build_stl_str_hl(wp, stcp->text, MAXPATHL, stc, "statuscolumn", OPT_LOCAL, ' ', + stcp->width, &stcp->hlrec, fillclick ? &clickrec : NULL, stcp); xfree(stc); // Only update click definitions once per window per redraw @@ -909,7 +906,7 @@ int build_statuscol_str(win_T *wp, linenr_T lnum, long relnum, int maxwidth, int stl_clear_click_defs(wp->w_statuscol_click_defs, wp->w_statuscol_click_defs_size); wp->w_statuscol_click_defs = stl_alloc_click_defs(wp->w_statuscol_click_defs, width, &wp->w_statuscol_click_defs_size); - stl_fill_click_defs(wp->w_statuscol_click_defs, clickrec, buf, width, false); + stl_fill_click_defs(wp->w_statuscol_click_defs, clickrec, stcp->text, width, false); } return width; |