Apache9 commented on PR #4312:
URL: https://github.com/apache/hbase/pull/4312#issuecomment-1102790308
The eclipse formatter can not express the above things exactly, let me show
you some examples.
This is 'Wrap where necessary'
```
class Example {
void foo() {
Other.bar(100, nested(200, 300, 400,
500, 600, 700, 800, 900));
}
}
```
This is 'Wrap first element, others where necessary'
```
class Example {
void foo() {
Other.bar(
100,
nested(
200, 300, 400, 500, 600, 700,
800, 900));
}
}
```
This is 'Wrap all elements, every element on a new line'
```
class Example {
void foo() {
Other.bar(
100,
nested(
200,
300,
400,
500,
600,
700,
800,
900));
}
}
```
This is 'Wrap all elements, indent all but first element'
```
class Example {
void foo() {
Other.bar(
100,
nested(
200,
300,
400,
500,
600,
700,
800,
900));
}
}
```
This is `Wrap all elements, except first elemen if not necessary`
```
class Example {
void foo() {
Other.bar(100,
nested(200,
300,
400,
500,
600,
700,
800,
900));
}
}
```
For me I think either we choose the first one, or the last one. The last one
will have more lines but I think it is more clear if we have lots of arguments.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
