Fix fold_handler method signature

[Nicholas-Hein]

Corrects the fold_handler LSP handler signature to conform with a
breaking NeoVim nightly update. This closes #5
by ensuring LSP results and buffer numbers are correctly gathered from
arguments.

[pierreglaser]

Thanks for the patch! Is there an easy way to remain backward-compatible with neovim versions preceding this change?

[ElPiloto]

Has enough time passed that we can switch to this despite it not being backwards-compatible?

[ajitid]

Hey @pierreglaser, I concur with @ElPiloto that we should merge this.

[Lazerbeak12345]

Thanks for the patch! Is there an easy way to remain backward-compatible with neovim versions preceding this change?

It should be possible to detect the neovim version to do this. Not the best idea since feature detection is always more reliable than version detection, but that's the only way I can think of.

[pierreglaser]

@ElPiloto @ajitid wasn't this resolved by 5d2b3d9?

[ajitid]

I don't remember, really. I look at the diff and the made the change in my own fork: ajitid@71e46d2. Seems like the change results in completely skipping inner if condition.

Unrelated, but another change I made is to only attach to those clients into which I've specified ajitid@6de3bb5. This reduces the nagging on upon opening each file.

[pierreglaser]

Unrelated, but another change I made is to only attach to those clients into which I've specified ajitid@6de3bb5. This reduces the nagging on upon opening each file.

Ah, I agree, being able to specify which client folding-nvim should ask folds to would be nice to have. PR welcome!

[pierreglaser]

I'm closing this for now (see #6 (comment)), but feel free to re-open if I missed something.