perf: reduce call api nvim_win_get_width 💪

I just realize we usually call nvim_win_get_width function on component
so i call it from render and send to component by params

benchmark
- evil_line reduce from 0.13 to 0.125
- airline reduce from 0.23 to 0.21

Author: windwp

README.md

@@ -196,7 +196,7 @@ basic.lsp_diagnos = {
         yellow = { 'yellow', 'black' },
         blue = { 'blue', 'black' },
     },
-    text = function(bufnr, winnr)
+    text = function(bufnr, winnr, width)
         if lsp_comps.check_lsp() then
             return {
                 -- `red` is define in hl_colors or a hightlight group name

lua/windline/component.lua

@@ -84,16 +84,16 @@ function Comp:setup_hl(colors)
 end
 
 
-function Comp:render(bufnr, winnr)
+function Comp:render(bufnr, winnr, width)
     self.bufnr = bufnr
     local hl_data = self.hl_data or {}
-    local childs = self.text(self.bufnr, winnr)
+    local childs = self.text(bufnr, winnr, width)
     if type(childs) == 'table'then
         local result = ''
         for _,child in pairs(childs) do
             local text,hl = child[1],child[2]
             if type(text) == 'function' then
-                text = child[1](bufnr, winnr)
+                text = child[1](bufnr, winnr, width)
             end
             if type(hl) == 'string' then
                 hl = hl_data[hl] or hl

lua/windline/init.lua

@@ -19,20 +19,15 @@ local mode = utils.mode
 
 M.statusline_ft = {}
 
-local is_width_valid = function (width, winnr)
-    if width == nil then return true end
-    return api.nvim_win_is_valid(winnr)
-        and width < api.nvim_win_get_width(winnr)
-end
-
 local render = function(bufnr, winnr, items, cache)
     M.state.comp = {} --reset component data
     M.state.mode = mode()
     Comp.reset()
     local status = ''
+    local win_width = api.nvim_win_is_valid(winnr) and api.nvim_win_get_width(winnr)
     for _, comp in pairs(items) do
-        if is_width_valid(comp.width, winnr) then
-            status = status .. comp:render(bufnr, winnr)
+        if win_width and (comp.width == nil or comp.width < win_width) then
+            status = status .. comp:render(bufnr, winnr, win_width)
         end
     end
     if cache then
@@ -268,8 +263,8 @@ M.benchmark = function()
     local bench = require('plenary.profile').benchmark
     local statusline = ''
     local time = bench(num, function()
-        vim.g.statusline_winid = vim.api.nvim_get_current_win()
-        statusline = WindLine.show(vim.api.nvim_get_current_buf(), vim.g.statusline_winid)
+        vim.g.statusline_winid = api.nvim_get_current_win()
+        statusline = WindLine.show(api.nvim_get_current_buf(), vim.g.statusline_winid)
     end)
     local popup = require('plenary.popup')
     local result = {}
@@ -279,18 +274,19 @@ M.benchmark = function()
     table.insert(result, string.format('render %s time : *%s*', num, time))
     table.insert(result, 'Comp:')
     local line = M.get_statusline_ft(vim.bo.filetype) or M.default_line
-    local bufnr = vim.api.nvim_get_current_buf()
-    local winnr = vim.api.nvim_get_current_win()
+    local bufnr = api.nvim_get_current_buf()
+    local winnr = api.nvim_get_current_win()
+    local width = api.nvim_win_get_width(0)
     table.insert(result, string.format('%s %12s %12s %s %s', ' ', 'time', 'name', 'num   ', 'text'))
     for index, comp in ipairs(line.active) do
         local item=''
         time = bench(num, function()
-            item = comp:render(bufnr, winnr)
+            item = comp:render(bufnr, winnr, width)
         end)
         table.insert(result, string.format('%02d *%10s* %12s %s - %s', index, time, comp.name or '   ', num, item))
     end
-    local width = math.floor(vim.o.columns / 1.5)
-    local col = math.floor((vim.o.columns - width) / 2)
+    local vim_width = math.floor(vim.o.columns / 1.5)
+    local col = math.floor((vim.o.columns - vim_width) / 2)
     popup.create(result, {
         border = {},
         minheight = 30,

lua/wlsample/airline.lua

@@ -62,14 +62,11 @@ airline_colors.c = {
 basic.divider = { b_components.divider, hl_list.Normal }
 
 local width_breakpoint = 100
-local check_width = function()
-    return vim.fn.winwidth(0) > width_breakpoint
-end
 
 basic.section_a = {
     hl_colors = airline_colors.a,
-    text = function()
-        if check_width() then
+    text = function(_,_,width)
+        if width > width_breakpoint then
             return {
                 { ' ' .. state.mode[1] .. ' ', state.mode[2] },
                 { sep.right_filled, state.mode[2] .. 'Sep' },
@@ -86,9 +83,9 @@ local get_git_branch = git_comps.git_branch()
 
 basic.section_b = {
     hl_colors = airline_colors.b,
-    text = function()
+    text = function(_,_, width)
         local branch_name = get_git_branch()
-        if check_width() and #branch_name > 1 then
+        if width > width_breakpoint and #branch_name > 1 then
             return {
                 { branch_name , state.mode[2] },
                 { ' ', '' },
@@ -128,8 +125,8 @@ basic.section_x = {
 
 basic.section_y = {
     hl_colors = airline_colors.b,
-    text = function()
-        if check_width() then
+    text = function(_,_,width)
+        if width > width_breakpoint then
             return {
                 { sep.left_filled, state.mode[2] .. 'Sep' },
                 { b_components.file_type({ icon = true }), state.mode[2] },
@@ -142,8 +139,8 @@ basic.section_y = {
 
 basic.section_z = {
     hl_colors = airline_colors.a,
-    text = function()
-        if check_width() then
+    text = function(_,_,width)
+        if width > width_breakpoint then
             return {
                 { sep.left_filled, state.mode[2] .. 'Sep' },
                 { '', state.mode[2] },
@@ -154,6 +151,7 @@ basic.section_z = {
         end
         return {
             { sep.left_filled, state.mode[2] .. 'Sep' },
+            { ' ', state.mode[2] },
             { b_components.line_col, state.mode[2] },
         }
     end,

lua/wlsample/evil_line.lua

@@ -67,8 +67,8 @@ basic.file = {
         white = { 'white', 'black' },
         magenta = { 'magenta', 'black' },
     },
-    text = function(_, winnr)
-        if vim.api.nvim_win_get_width(winnr) > breakpoint_width then
+    text = function(_, _, width)
+        if width > breakpoint_width then
             return {
                 { b_components.cache_file_size(), 'default' },
                 { ' ', '' },
@@ -95,8 +95,8 @@ basic.file_right = {
         white = { 'white', 'black' },
         magenta = { 'magenta', 'black' },
     },
-    text = function(_, winnr)
-        if vim.api.nvim_win_get_width(winnr) < breakpoint_width then
+    text = function(_, _, width)
+        if width < breakpoint_width then
             return {
                 { b_components.line_col, 'white' },
                 { b_components.progress, '' },