|
From: | Bozhidar Batsov |
Subject: | Re: [ruby-mode] dot-alignment support for multi-line method chaining |
Date: | Wed, 22 Jan 2014 19:28:51 +0200 |
Hi!
Good suggestion, but it looks harder than the previous ones, and it'll
Bozhidar Batsov <address@hidden> writes:
> Currently when doing method chaining on multiple lines we get the
> following indentation:
>
> something.ala
> .one
> .two
> .three
>
> An alternative style that seems to be popular is:
>
> something.ala
> .one
> .two
> .three
require modifying (or maybe "fixing") the SMIE grammar. At the moment,
the parent of a "." token is never the previous ".".
I have an experimental patch, included at the end, but it throws up some
grammar priority warnings, and it changes the indentation in the
following example:
zoo.keep.bar!(
{x: y,
z: t})
If I modify the code a little, the indentation here will be like this:
zoo.keep.bar!(
{x: y,
z: t})
which corresponds to
zoo.keep.bar!(
{x: y,
z: t}
)
which, in turn, is a small modification of
zoo.keep
.bar!(
{x: y,
z: t}
)
Would you consider this appropriate, for the "fluent interface"
indentation? Or how would you indent it instead?
By the way, do you think it'd be good if the same variable toggled between
foo = foobar
.baz
and
foo = foobar
.baz
? Or would that be a different variable?
P.S. Could you use the bug tracker for feature requests next time? Bugs
are easier to keep track of and to get back to later.
=== modified file 'lisp/progmodes/ruby-mode.el'
--- lisp/progmodes/ruby-mode.el 2014-01-17 03:15:02 +0000
+++ lisp/progmodes/ruby-mode.el 2014-01-22 13:25:38 +0000
@@ -351,7 +351,7 @@
(exp "and" exp) (exp "or" exp))
(exp (exp1) (exp "," exp) (exp "=" exp)
(id " @ " exp)
- (exp "." id))
+ (id "." exp))
(exp1 (exp2) (exp2 "?" exp1 ":" exp1))
(exp2 ("def" insts "end")
("begin" insts-rescue-insts "end")
@@ -380,7 +380,7 @@
(ielsei (itheni) (itheni "else" insts))
(if-body (ielsei) (if-body "elsif" if-body)))
'((nonassoc "in") (assoc ";") (right " @ ")
- (assoc ",") (right "=") (assoc "."))
+ (assoc ",") (right "="))
'((assoc "when"))
'((assoc "elsif"))
'((assoc "rescue" "ensure"))
@@ -399,7 +399,8 @@
(nonassoc ">" ">=" "<" "<=")
(nonassoc "==" "===" "!=")
(nonassoc "=~" "!~")
- (left "<<" ">>"))))))
+ (left "<<" ">>")
+ (assoc "."))))))
(defun ruby-smie--bosp ()
(save-excursion (skip-chars-backward " \t")
@@ -622,7 +623,10 @@
(unless (or (eolp) (forward-comment 1))
(cons 'column (current-column)))))
(`(:before . "do") (ruby-smie--indent-to-stmt))
- (`(:before . ".") ruby-indent-level)
+ (`(:before . ".")
+ (if (smie-rule-sibling-p)
+ 0
+ ruby-indent-level))
(`(:after . "=>") ruby-indent-level)
(`(:before . ,(or `"else" `"then" `"elsif" `"rescue" `"ensure"))
(smie-rule-parent))
[Prev in Thread] | Current Thread | [Next in Thread] |