indent explicitly set in multiline tqs

Hi Barry,

AFAIU tqs is not at stake here.

Would describe the rule in effect as:

"if inside a list indent is set explicitly, keep that"

Resp. would rename the report into something like:

"keep indent inside lists"

Cheers,

Andreas

Hey Andreas,

can you explain your previous comment a bit more? I don't see a list involved here...

well, 'bar' opens an args-list.

Wrong?

OTOH:

seems we have two bugs at once in this example:

the "explicite" issue

tqs-indent should be honored if inside an list (?)

Oh, ok, I wasn't thinking of argument lists.

I agree about keeping explicitly selected indentation.

Andreas, I think you're right but I'm not positive. I do know that "old" python-mode.el mostly got this right, so it might be worth going back a bunch of revisions and doing some experimentation.

On 18.05.2011 14:00, Barry Warsaw wrote:
> Andreas, I think you're right but I'm not positive.

Hi Barry,

AFAIS we agree with respect to intended --ie. optional possible--
behaviour at this point.

It's just to describe the rule - for clarification for me, what to do
and why.

What about that formula:

"When inside an multiline list an element is indented explicitely,
follow that indent". (?)

Cheers,

Andreas

  I do know that
> "old" python-mode.el mostly got this right, so it might be worth going
> back a bunch of revisions and doing some experimentation.
>

Hi Andreas, that certainly seems like a reasonable rule to me.

On 18.05.2011 16:58, Barry Warsaw wrote:
> Hi Andreas, that certainly seems like a reasonable rule to me.
>

OK, thanks Barry.

I'll rename the report than pointing at "list" in it's name.

Might exist a related TQS issue though.

In this case, think it's helpful to get a separate report than.

Cheers,

Andreas

Hi Barry,

after some further reflexion:

being inside a list isn't of interest either: it's just the multiline character of expression, which matters.
Must consider backslashed expressions too.

Also it's not just dedent, but any indent explicitly set.

The rule now reads:

"If inside a multiline expression indent is set explicitly, honor it."

Will change the name accordingly.

Cheers,

Andreas

Hi Barry,

if inside a tqs or not matters with respect to implementation/tests.
Correcting my previous post so far.

In order to provide case-tests will write another report for non-tqs issues.

Cheers,

Andreas