From 83925be1e68445f7f6a377b7735f93fbc8689003 Mon Sep 17 00:00:00 2001 From: Dylan Thacker-Smith Date: Wed, 21 Feb 2024 21:03:10 +0100 Subject: patch 9.1.0125: unused init of text_prop_idxs in win_line() Problem: unused init of text_prop_idxs in win_line Solution: Remove it, fix typo (Dylan Thacker-Smith) Later use of text_prop_idxs treats it as empty, incrementing text_props_active as new elements are added to this array, so remove this unused conditional initialization when text_props_active is 0. closes: #14063 Signed-off-by: Dylan Thacker-Smith Signed-off-by: Christian Brabandt --- src/drawline.c | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) (limited to 'src/drawline.c') diff --git a/src/drawline.c b/src/drawline.c index 0f3400f9da..3499d39e77 100644 --- a/src/drawline.c +++ b/src/drawline.c @@ -1633,16 +1633,6 @@ win_line( if (text_props[i].tp_id < 0) last_textprop_text_idx = i; - // When skipping virtual text the props need to be sorted. The - // order is reversed! - if (lnum == wp->w_topline && wp->w_skipcol > 0) - { - for (int i = 0; i < text_prop_count; ++i) - text_prop_idxs[i] = i; - sort_text_props(wp->w_buffer, text_props, - text_prop_idxs, text_prop_count); - } - // Text props "above" move the line number down to where the text // is. Only count the ones that are visible, not those that are // skipped because of w_skipcol. @@ -2298,7 +2288,7 @@ win_line( // If another text prop follows the condition below at // the last window column must know. - // If this is an "above" text prop and 'nowrap' the we + // If this is an "above" text prop and 'nowrap' then we // must wrap anyway. text_prop_above = above; text_prop_follows |= other_tpi != -1 -- cgit v1.2.3