michaelrommel / nvim-silicon

neovim plugin to create code images using the external silicon tool.
MIT License
110 stars 8 forks source link

line_offset function receives null args #15

Closed malcolmrebughini closed 4 months ago

malcolmrebughini commented 4 months ago

Hi!

The documentation in the readme states the following:

-- with which number the line numbering shall start
    line_offset = 1,
    -- here a function is used to return the actual source code line number
    -- line_offset = function(args)
    --  return args.line1
    -- end,

But using a function leads to the following error due to the args being nil:

14:26:14 msg_show   '<,'>Silicon Error executing Lua callback: .../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:48: Vim:Error executing Lua callback: /home/malcolm/.config/nvim/lua/user/plugins/user.lua:166: attempt to index local 'args' (a nil value)
stack traceback:
    /home/malcolm/.config/nvim/lua/user/plugins/user.lua:166: in function 'v'
    ....local/share/nvim/lazy/nvim-silicon/lua/silicon/init.lua:78: in function 'get_arguments'
    ....local/share/nvim/lazy/nvim-silicon/lua/silicon/init.lua:142: in function 'start'
    ....local/share/nvim/lazy/nvim-silicon/lua/silicon/init.lua:236: in function <....local/share/nvim/lazy/nvim-silicon/lua/silicon/init.lua:235>
    [C]: in function 'cmd'
    .../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:48: in function <.../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:16>
stack traceback:
    [C]: in function 'cmd'
    .../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:48: in function <.../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:16>

Neovim version: neovim 0.9.5-2

michaelrommel commented 4 months ago

I am sorry about that, it should be fixed in the last version. I omitted an argument in the last refactor. Shame on me... Please try again with the latest version and close the issue, if it is fixed for you!

malcolmrebughini commented 4 months ago

@michaelrommel thanks for the quick reply and fix! Works perfectly now. Closing this issue.