[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