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

Sat Jan 21 20:13:01 PST 2023

owenpan added a comment.

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

> In D141654#4053470 <https://reviews.llvm.org/D141654#4053470>, @rymiel wrote:
>
>> This also made me wonder, is there an actual policy on deprecated options? As in, when they are actually removed. Is there even prior precedent for doing this? (I wouldn't know because I'm a very recent user)
>
> I think the first one was D108752 <https://reviews.llvm.org/D108752> and the documentation can handle that since D127578 <https://reviews.llvm.org/D127578>.
> Removing an option is not different than changing the type of an option, users of `libformat` have to adapt, and users of `clang-format` don't notice (if everything is done right), because the parsing for the removed options is still there, see lines 1068 and following of `Format.cpp` in this change.

+1.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141654