Support customizing JS function finder

[somini]

This adds a way to show only the function name in the Javascript cfi.

I think this could be optimized, but the original code is fast, no harm done.

[tyru]

@somini Hmm, there is already similar global variable g:cfi_javascript_show_assignment. How about adding g:cfi_javascript_show like this?

let g:cfi_javascript_show = ["assignment", "varname", "funtype", "funname", "funargs", "funbody"]

g:cfi_javascript_show_assignment should be also deprecated if this variable is merged.

[somini]

Yeah, it's not sustainable to just have a bunch of one-off switches.

I'll follow your advice and implement that.

[tyru]

Thanks! :)

[tyru]

ah, maybe Dictionary is preferrable rather than List.

• For user side: easy to set/remove key-value.
• For cfi.vim implementation side: easy to merge user value with default one by extend().

let g:cfi_javascript_show = {"assignment": 1, "varname": 1, "funtype": 1, "funname": 1, "funargs": 1, "funbody": 1}

[somini]

Here's a first implementation. I'll test this tomorrow.

[somini]

I've fixed this.

This has a lot of junk commits, I'll squash this into a single commit and force push to this PR.

[tyru]

This has a lot of junk commits, I'll squash this into a single commit and force push to this PR.

Okay, no problem. However I don't care if there are a lot of commits in PR :)

[tyru]

@somini is it okay to merge this?

[somini]

Yes, I've been using it regularly and haven't run into any issues yet. If anyone opens new issues related to this, please ping me.

[tyru]

This has a lot of junk commits, I'll squash this into a single commit and force push to this PR.

How is this going?

[tyru]

I also feel this PR has no problems. I was just waiting you squash this. (I'm okay if you choose you don't do that)

[somini]

It's already 1 commit only, you can merge it.

[tyru]

@somini Okay, now merged it! Thanks!