[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#20896: patch to add chained indentation

From: Dmitry Gutov
Subject: bug#20896: patch to add chained indentation
Date: Thu, 12 Jan 2017 04:57:49 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:50.0) Gecko/20100101 Thunderbird/50.0

On 10.01.2017 09:29, Tom Tromey wrote:
This patch adds chained indentation, as requested in the bug.


It comes
with some tests

I'd just like to point out that it's much better to write indentation tests in the format used by test/manual/indent/js*.

(added to a file that first appears in patch in another

Not sure which patch you mean. This file seems new.

As for review: js--skip-term-backward seems to be doing something similar to the loop in js--multi-line-declaration-indentation.

Maybe a extraction and unification is in order.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]