[PATCH] D127390: [clang-format][NFC] Remove unused FormatStyle members

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jun 11 12:13:07 PDT 2022


owenpan added a comment.

In D127390#3575628 <https://reviews.llvm.org/D127390#3575628>, @MyDeveloperDay wrote:

> If we remove it from here, then it will get removed from  the documentation
>
> F23403281: image.png <https://reviews.llvm.org/F23403281>
>
> Also if you remove it, then the ClangFormatStyleOptions.rst needs to be regenerated.

Good catch! I will regenerate it.

> But somehow I think we need that comment no? incase someone is looking up their legacy options when migrating?

IIRC there were other deprecated options that were removed from Format.h. These two options were old enough and IMO we left them in Format.h long enough. Nevertheless, I will see if we can add the comments back without keeping the `FormatStyle` members.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127390



More information about the cfe-commits mailing list