load function does not correctly load nested function via : ,ff about nvim-r HOT 6 CLOSED

Please, review you post. It seems that there is something wrong with the markdown format.

What happens if you end the function with

    }
}

instead of

    }}

from nvim-r.

If I copied your code correctly, I think <LocalLeader>ff is getting the function correctly. It does not rely on indentation, and it is getting the nested function completely. If you want to send the outer function to R, you should either put the cursor outside the nested function or mark the block of code and press <LocalLeader>bb.

from nvim-r.

Correct, but since the inner function is part of the lexical scope for the outer function, then the inner function should only be reload in context of the outer function. Not in the global scope.

A similar case is when you have a tryCatch function, where you have defined error, warning, interrupt function. You need to move to the outer function outside of the tryCatch to reload/ otherwise you get an compile error.

from nvim-r.

Certainly you are right for most use cases, but in some circumstances the user might want to test only the inner function. Anyway, it would be too complicated to implement a smart algorithm that decides what to send to R based on context. Unless someone is wanting to implement this, it might be better to remove the functionality triggered by <LocalLeader>ff than having the currently broken one.

We already have an alternative in the plugin: to send a marked block of lines. The user has only to press, for example, ma before the lines that must be sent and mb after them to mark the block and, then, press <LocalLeader>bb. In the Nvim-R documentation I suggest the use of vim-signature plugin to make the marks visible.

from nvim-r.

I would probably keep the current imp; maybe change the documentation to indicate it only loads current function scope.

I could probably try to cook something together like <localleader>ft (reload toplevel function in scope).

I think I have a hackup unction somewhere that does it for clojure.

from nvim-r.

I added a note on this other limitation to section 5.3 (6428766).

from nvim-r.