[PATCH] D88700: [clang-tidy] modernize-use-trailing-return-type fix #44206

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 3 08:11:44 PDT 2020


aaron.ballman added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/modernize/UseTrailingReturnTypeCheck.cpp:72
+                TL.getAs<TypedefTypeLoc>().getTypePtr()->getDecl()->getName()))
+          return false;
       default:
----------------
bernhardmgruber wrote:
> aaron.ballman wrote:
> > bernhardmgruber wrote:
> > > There is a `break;` missing after this line. The build bot just told me. It should not change the meaning of the program though.
> > > 
> > > Can someone add that for me or do I need to upload another diff? Thank you!
> > I'll take care of it, thanks for letting me know!
> Great! Thank you for the help!
Any time! The fix is in 089e628b61f929ccd26565cd4118395f0a0273c3


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88700



More information about the cfe-commits mailing list