Newlines after functions about riml HOT 5 CLOSED

I made a pass at doing this here.

The change itself was easy, Adding a newline to node.compiled_output, but a lot of tests depend on the newline being present when comparing against compiled code. The test suite fails, but the compiler works fine and the output formatting is improved, as seen here.

I could add the newlines to the tests. But given the number of tests(~50) that do this equality test I felt it better to get your thoughts on this first.

https://github.com/dsawardekar/riml/tree/improve-output-formatting

from riml.

Hey, sorry for delayed reply. Busy week 😦

Yeah, definitely a small change, and I'm 👍 on it. Maybe there should be an option, like --readable for this though, as I was going to create a similar option for --keep-comments. This would allow for keeping the fold markers, modelines, etc...

Not sure if --readable should be the default option, but if you find yourself digging into the outputted VimL often then I may as well make it the default.

I'd like your thoughts on if you think this should be default behavior.

Thanks!

from riml.

I haven't needed to drop into Viml too much, thanks to Riml! But it's useful when doing some meta programming to figure out the code generated.

The --readable sounds right to me, and I agree it should be default. --keep-comments is pretty good too.

While we are on the subject of comments, would it be a difficult change to add support for a different comment marker like #? Viml use of quote as the comment marker makes the syntax files regex a lot more complex, and messes up the syntax highlighting often...

from riml.

Blank lines after function definitions is the new default in 0.3.2. This can be turned off using the --condensed flag. My first go at --keep-comments failed, because preserving inline comments after code is actually pretty hard the way everything is implemented now. Keeping comments that are on their own lines should be easy enough. Anyway, I'm marking this as done, and I'll add another ticket so I don't forget to eventually come back to the comment stuff. As it stands, I think better error messages are probably a priority over keeping comments in the compiled files.

Thanks 😄

from riml.

+1 For better error messages!

from riml.