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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to