Not a problem for me, but could you test the attached patch anyway?
It seems to handle a bunch of different/complex cases fine without
regressions, but it's always better with a second pair of eyes.
Sure. I tried a few things and the only problem I can find is that it
does not handle endless module methods:
def self.some_method =
"some-value"
(with or without the line break, it handles them as the unpatched
version handles instance endless methods.
I'd probably use ruby-mode if
it supported indenting long parameter/argument lists the way the
non-smie version does, like this:
def some_method(
some_param,
some_other_param
)
Now that the SMIE stuff is again in my short-term memory, it shouldn't
be too hard. Just please file a separate bug report (slash feature
request) with a precise example. Bonus points for linking to a relevant
Rubocop rule, so that we can pick a better name for the new user option.
Sure thing, just sent one in: bug#60110
I don't see the non-SMIE version indenting it like this -- it looks more
like this instead (and only if I set ruby-deep-indent-paren to nil):
def test2 (
asd,
asd
asd
)
So let's start with a couple of good examples.
Yeah, this is what I see too. Not sure what I saw before. In any case,
I sent a current/desired.
ruby-endless-methods.diff