Scope of variables becomes scriptlocal inside elseif

[luke-gru]

Scope of variables becomes scriptlocal inside elseif

[luke-gru]

Thanks for the detailed issue report! This is fixed in the new version I just released, 0.3.0. It had to do with not putting the ElseifNode's condition expression in the children of the node, woops! 😄

This new release also contains the default class scoping to 's:' (class_scopes branch), along with another branch I merged in called private_funcs, that makes private functions (defined with def, not defm) a lot better to work with. You can check an example of a private function in the CHANGELOG. I'll be adding docs to the README as well shortly.

Oh, and as for the release process, I'll be merging in your PR in a bit.

Thanks again!

[dsawardekar]

Awesome! I upgraded Speckle to 0.3.0. Everything went smoothly. I needed to use g: for code that I'm evaling. And I removed some of the `l:' workarounds in elseifs. All those bugs fixes worked great. :)

I'm pleased to report that the switch to default s: scope for classes did not cause any trouble. I didn't even need change any of my specs! Also thanks for fixing the splats bug. I was going to report that too, but didn't want to impose too much. :)

The private scoped def looks neat. Prefixing <SID> should make it pretty safe. Though I'm sure some Vimscript Jedi master is going to figure a way around it!