[PATCH] D54222: [clang-tidy] Add a check to detect returning static locals in public headers
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 8 15:44:44 PST 2018
aaron.ballman added inline comments.
================
Comment at: docs/clang-tidy/checks/llvm-problematic-statics.rst:7
+
+Looks for functions defined in "public" headers that return the address of a static local. These are not guaranteed to have the addresss across shared libraries and could be problematic for plugins.
+
----------------
Eugene.Zelenko wrote:
> static local variable? Probably //Looks for// should be replaced with //Detects//. Please also wrap with 80 characters.
to have the address -> to have the same address
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D54222
More information about the cfe-commits
mailing list