[Python-mode] one more myrkwid bug

[Barry Warsaw]

Hi Andreas,

On Apr 15, 2011, at 08:59 AM, Andreas Röhler wrote:

>def foo():
>     x = dict(
>              a=1,
>              b=2,
>              c=3,
>              )
>
>
>which looks okay for me.

Except it's a regression from the trunk.  When the open paren ends a line,
subsequent lines should get indented only one level, not to the space after an
open paren.  At least, that's the way it should work by default.

>If "4 spaces after assignement start" shall be the rule, we get next bug
>report, should someone leave out the spaces, which would produce a nasty
>
>def foo():
>     x=dict(
>         a=1,
>         b=2,
>         c=3,
>         )

Note that it doesn't have anything to do with the assignment, or the dict.
It's the open paren ending a line that's the critical thing here.  Here's
another example.  This is what myrkwid gives you:

-----snip snip-----
def foo():
    do_something_first(
                       a=1,
                       b=2,
                       c=3,
                       )
-----snip snip-----

But this is what trunk gives you:

-----snip snip-----
def foo():
    do_something_first(
        a=1,
        b=2,
        c=3,
        )
-----snip snip-----

A further problem with myrkwid is that if I'm typing the first few lines:

-----snip snip-----
def foo():
    do_something_first(
                       a=1,
-----snip snip-----

and now I realize that 'a=1' is indented too far to the right, so I manually
correct it to start under the 'o' in 'something'.  Then I go to the end of the
line and hit return.  I end up with this, which is definitely not correct:

-----snip snip-----
def foo():
    do_something_first(
        a=1,
                       b=2,
-----snip snip-----

'b' should line up under 'a'.

See what I mean?

>Would make that customizable with a var
>
>align-to-dict
>
>or so, which would be true here, nil the default. (?)

>BTW: indent should go along with meaning, if possible.
>Well, you being the authority :)

Or just the most adventurous and vocal, anyway :).  Indentation controlling
variables should be well-named and discoverable, but you just have to be
careful to capture the right semantics.  In this case, the indentation has
nothing to do with assignments or dicts, but everything to do with lines that
end in open parens.

Hope that helps!

>PS Applied patch from 691542 this morning. Will push it into myrkwid also.

Awesome, thanks!
-Barry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://mail.python.org/pipermail/python-mode/attachments/20110415/3e809384/attachment.pgp>