[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