Subject: bug#16118: closed (Re: bug#16118: Acknowledgement (24.3.50; ruby-smie-rules: hanging literal in first method call argument is indented 1 column too few))

From: [email-masked] (GNU bug Tracking System)
Date: Sat, 14 Dec 2013 06:50:07 +0000

Your bug report

#16118: 24.3.50; ruby-smie-rules: hanging literal in first method call argument is indented 1 column too few

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
16118: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=16118
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#16118: Acknowledgement (24.3.50; ruby-smie-rules: hanging literal in first method call argument is indented 1 column too few)
From: Dmitry Gutov <[email-masked]>
To: [email-masked]
Date: Sat, 14 Dec 2013 08:49:46 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Version: 24.4

Revision 115514.

Subject: 24.3.50; ruby-smie-rules: hanging literal in first method call argument is indented 1 column too few
From: Dmitry Gutov <[email-masked]>
To: [email-masked]
Date: Thu, 12 Dec 2013 07:48:41 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

We have this example in indent/ruby.rb:

foo({
     a: b,
     c: d
   })

- The curlies don't line up.
- If I expand this example, the hash literal in the first argument
  doesn't line up with the arguments after it:

foo({
     a: b,
     c: d
   },
    {
      e: f
    })

foo({
     a: b,
     c: d
   }, :bar => {
      e: f
    })

foo(:bar =>
    {
      a: b,
      c: d
    }, :tee =>
       {
      e: f
    })

...but:

foo({
     a: b,
     c: d
   }, {
     e: f
   })

Still, I think the literals should be lined up to the first column after
"(", not the column of "(" itself. How can we do that?

Adding a `(1+ )' in the relevant place in `ruby-smie-rules' is an
obvious solution, but it breaks other examples.

In GNU Emacs 24.3.50.8 (x86_64-unknown-linux-gnu, GTK+ Version 3.8.6)
 of 2013-12-09 on axl
Bzr revision: 115440 [email-masked]
Windowing system distributor `The X.Org Foundation', version 11.0.11403000
System Description:     Ubuntu 13.10