Why not make SlimeSend public?

[moll]

Hey,

I'm trying to use Slime to start a process straight from Vim, and for that it'd be really handy to just send a command line down to Screen. I see there's the SlimeSend function which is exactly what I'm after, but it's private. Why not make it public?

[jpalardy]

I think I didn't expose the function because I didn't think it was useful.

If you patch it, test it, and send a pull request — I'll accept it :-)

[moll]

I assume by testing you mean manual, because there are no existing unit tests for vim-slime? :P
But I'll give it a spin hopefully in the near future.

There is an existing command titled SlimeSend which actually calls SlimeSendRange or something. I'm wondering how to handle that naming conflict — SlimeSend would make more sense to be the name of the command that you can give a string argument to.

[jpalardy]

Consider SlimeSend to be the public interface, while SlimeSendRange is a helper.

[moll]

^ Closing makes sense for now. I'll ping back when I get around to working on this! ;)

[jpalardy]

Sure thing.

[moll]

I'm guessing #36 did that work already. Yay for others doing the work. 8-)