[PATCH] D53072: [clang-format] Introduce the flag which allows not to shrink lines

Ivan Donchevskii via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 7 03:43:39 PST 2019


yvvan added a comment.

In D53072#1342363 <https://reviews.llvm.org/D53072#1342363>, @djasper wrote:

> I don't quite understand. What you are describing should already be the behavior with ColumnLimit=0 and I think your test should pass without the new option. Doesn't it?


As far as I see how it works column limit does not prevent the shrinking behavior.

In D53072#1342353 <https://reviews.llvm.org/D53072#1342353>, @krasimir wrote:

> In D53072#1268883 <https://reviews.llvm.org/D53072#1268883>, @yvvan wrote:
>
> > Do you know the better way to accomplish my aim than adding an option to libFormat? For example making a dependent library which serves a little different purpose than libFormat itself or something simpler?
>
>
> I don't know about the specific use-case, but in general a fail-safe way to force a line break in C++ code is to add an empty line comment on the preceding line.


This is close but will not work for the case when more than two lines are packed together. So it requires to put the empty comment on the each line ending which is not too nice (if i'm not missing something)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D53072/new/

https://reviews.llvm.org/D53072





More information about the cfe-commits mailing list