[PATCH] D76572: Replace `T(x)` with `reinterpret_cast<T>(x)` everywhere it means reinterpret_cast. No functional change

Noel Grandin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 23 08:09:17 PDT 2020


grandinj added a comment.

In D76572#1936861 <https://reviews.llvm.org/D76572#1936861>, @Quuxplusone wrote:

> Nice. Does LibreOffice have anything (either in clang-tidy or in a paper guideline) against `T(x)`-style casts? E.g.


No, we don't have very many of those in our codebase, so we have left them alone.
Our plugin is designed to convert c-style casts to modern C++ casts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76572





More information about the cfe-commits mailing list