<html>
<head>
<base href="https://bugs.llvm.org/">
</head>
<body><table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Bug ID</th>
<td><a class="bz_bug_link
bz_status_NEW "
title="NEW - Trailing comments break AlignConsecutive* instead of being reflowed"
href="https://bugs.llvm.org/show_bug.cgi?id=46381">46381</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>Trailing comments break AlignConsecutive* instead of being reflowed
</td>
</tr>
<tr>
<th>Product</th>
<td>clang
</td>
</tr>
<tr>
<th>Version</th>
<td>unspecified
</td>
</tr>
<tr>
<th>Hardware</th>
<td>All
</td>
</tr>
<tr>
<th>OS</th>
<td>All
</td>
</tr>
<tr>
<th>Status</th>
<td>NEW
</td>
</tr>
<tr>
<th>Severity</th>
<td>enhancement
</td>
</tr>
<tr>
<th>Priority</th>
<td>P
</td>
</tr>
<tr>
<th>Component</th>
<td>Formatter
</td>
</tr>
<tr>
<th>Assignee</th>
<td>Jake.Merdich@amd.com
</td>
</tr>
<tr>
<th>Reporter</th>
<td>Jake.Merdich@amd.com
</td>
</tr>
<tr>
<th>CC</th>
<td>djasper@google.com, klimek@google.com, llvm-bugs@lists.llvm.org
</td>
</tr></table>
<p>
<div>
<pre>AlignConsecutive* takes place after comment reflowing, but one of the
conditions of breaking consecutive alignment is a line being too long. If a
line has a trailing comment that can be reflowed (or has already been reflowed)
but is otherwise too long, it would make more sense to still do the alignment
but reflow the comment afterwards to honor the column limit.
This would still need some mechanism to bail during consecutive alignment if
the comments cannot be reflowed without breaking the column limit and maybe
also for ridiculous cases like trailing comments with one word per line (a
penalty for breaking alignment would make sense).
This is a non-trivial change, so I'm leaving it here for tracking and feedback
while I think of a good way to do this cleanly.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>