Inconsistent jumps in jump action

[danymat]

Hello, I found some issues with this option:

• options mapping is not overridden by default: you need to add: u.merge(self.mapping, opts.mapping) in setup function.
• Given this file:

function P(...)
	local objects = vim.tbl_map(vim.inspect, { ... })
	print(unpack(objects))
	return ...
end

RELOAD = function(...)
	return require("plenary.reload").reload_module(...)
end

R = function(name, opts)
	opts = opts or {}
	RELOAD(name)
	if opts.setup then
		return require(name).setup(opts.setup)
	end

	return require(name)
end

Prequire = function(module)
	local ok, mod = pcall(require, module)
	if ok then
		return mod
	end
end

Wrap = function(function_pointer, ...)
	local params = { ... }

	return function()
		return function_pointer(unpack(params))
	end
end

And this config:

		use({
			"ZhiyuanLck/smart-pairs",
			event = "InsertEnter",
			config = function()
				require("pairs"):setup({
					mapping = {
						jump_left_in_any = "<C-h>",
						jump_right_out_any = "<C-l>",
					},
				})
			end,
		})

jump left seems to always go to the first (of the file, and jump right seems to miss some right brackets.
Nevertheless, it's a very good plugin, and I kinda love it !

[ZhiyuanLck]

Thanks for your feedback! All options has been merged now.

There is a plan to rewrite the jump action after next refactor.

[danymat]

Nice, thanks ! I keep forward to see the fix, good job !

[danymat]

There is a plan to rewrite the jump action after next refactor.

Hello @ZhiyuanLck, what's the state of the refactor ?

[ZhiyuanLck]

Sorry, last week I have to work on the conference paper whose deadline is 3.7. The refactor work will continue after the paper deadline.

[danymat]

No problem, I was asking you because I wanted to work on this issue, but if you do refactor then it's not useful because it'll be re-written