<div dir="ltr"><div dir="ltr">On Thu, Nov 30, 2023 at 5:17 PM Kevin Bourrillion <<a href="mailto:kevinb@google.com">kevinb@google.com</a>> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Of course it makes some maintenance tasks (i.e. reordering lists) strictly easier. But I don't always see it mentioned that it also leads to smaller diffs, which makes "blame" info slightly more accurate, merge conflicts slightly less common, and code reviews slightly more efficient. Of course, "slightly" is charitable; it's a very small effect. Still, in my non-Java "fun" projects I use them all the time and it seems strictly worse not to.</div></blockquote><div><br></div><div>My $0.03 (inflation)...</div><div><br></div><div>Your point about smaller diffs and reordering lists is correct, and (I'm guessing) what originally motivated this feature.</div><div><br></div><div>However, that motivation only applies when the items you are listing are normally or typically placed on separate lines.</div><div><br></div><div>That's true e.g. for enum constants, but not for the other stuff like method parameters, etc.<br></div><div><br></div><div>-Archie<br></div></div><br><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature">Archie L. Cobbs<br></div></div>