[PATCH] D141654: [clang-format] Replace DeriveLineEnding and UseCRLF with LineEnding

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 13 15:40:30 PST 2023


owenpan added a comment.

In D141654#4052510 <https://reviews.llvm.org/D141654#4052510>, @HazardyKnusperkeks wrote:

> In D141654#4050852 <https://reviews.llvm.org/D141654#4050852>, @rymiel wrote:
>
>> (This needs to run the updated dump script from D138446 <https://reviews.llvm.org/D138446>)
>>
>> I'm not sure what the strict //benefit// of squishing the two options together is, but I support it
>
> Having just one option instead of two, for one feature is better, especially when the two options are that far apart in the documentation.

+1.


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

https://reviews.llvm.org/D141654



More information about the cfe-commits mailing list