[erlang-patches] [erlang-bugs] erlang.el indenter

Fredrik fredrik@REDACTED
Wed Aug 22 12:14:17 CEST 2012


Hello Tomas,
We have included this patch in the 'pu' branch.
Thank you for the contribution!

BR Fredrik G
Erlang/OTP Team
On 08/21/2012 11:00 PM, Tomas Abrahamsson wrote:
> On Mon, Aug 6, 2012 at 7:42 PM, Tuncer Ayaz<tuncer.ayaz@REDACTED>  wrote:
>> There seems to be a bug in the indenter that mis-indents
>> rebar.erl:commands/0, if it's indented as part of a larger region.
>> The multi-line binary string seems to be the problem.
>>
>> rebar.erl:commands/0:
>> https://github.com/basho/rebar/blob/491d52298e2/src/rebar.erl#L275-311
>>
>> It works correctly when indenting the function itself with C-c C-q.
> Hi,
>
> Here is a patch for indenting of multi-line strings, like the
> one previously reported by Tuncer Ayaz.
>
> git fetch git://github.com/tomas-abrahamsson/otp.git
> emacs-indent-multi-line-string
>
> https://github.com/tomas-abrahamsson/otp/compare/emacs-indent-multi-line-string
> https://github.com/tomas-abrahamsson/otp/compare/emacs-indent-multi-line-string.patch
>
> Here's an illustration of what the patch does,
> excerpt from the commit msg:
>
>      Previously, typing C-M-\ or M-x indent-region RET on the following:
>
>          somefunction() ->
>              S =<<"
>          text1 somemore1
>          text2 somemore2
>          ">>,
>              io:put_chars(S).
>
>      resulted in this (note the "text2" line):
>
>          somefunction() ->
>              S =<<"
>          text1 somemore1
>                    text2 somemore2
>          ">>,
>              io:put_chars(S).
>
>      Now, the indentation inside the multi-line string is left unchanged.
>      This is in line with how the Emacs C and Lisp mode treats multi-line
>      strings when indenting regions.
>
>
> BRs
> Tomas
> _______________________________________________
> erlang-patches mailing list
> erlang-patches@REDACTED
> http://erlang.org/mailman/listinfo/erlang-patches




More information about the erlang-patches mailing list