some random style nits

most of these were suggested by intellij but pass a gut check to me; lmk if you there are conventions that we should decide more clearly one way or the other on.
I don't think it contradicts anything in scalariform.

Getting a linter for things like line length would probably be a good idea, generally, but that's another PR :)