#146 Indent: adding new method argument

[Charles Reis]

Type the following line:

public void foo(int a, int b)

Now assume you want to add a third argument, c. Type a
comma after b and hit enter. The close paren will move
to the next line with no indentation at all, until you
type any letter and hit tab:

public void foo(int a, int b,
)

If you just hit enter after b, the close paren is
indented "correctly".

[Tim Oberg]

Logged In: YES
user_id=691516

James and I have been working on this bug. The problem we
have found is that our fix breaks the method in IndentTest
that tests indenting anonymous inner classes passed to
methods. This is an issue we want to bring up in class
perhaps, as indentation for anonymous inner classes passed
to methods is currently broken in some cases anyways.

[Charles Reis]

Logged In: YES
user_id=429731

Yes, it's going to be tricky, and might require some other
form of context.

Could you post an example of an inner class that breaks
indentation now?

[Tim Oberg]

Logged In: YES
user_id=691516

see bug 685124

[Charles Reis]

Logged In: YES
user_id=429731

Raising the priority of this bug, since it is being
addressed as part of Task 70906 ("Indent Bugs").

[Charles Reis]

Logged In: YES
user_id=429731

Lowering the priority for the time being, since we are
moving on to other projects in 312.