Re: [PATCH] D22513: [clang-tidy] add check cppcoreguidelines-special-member-functions

2016-08-01 Thread Eric Lemanissier via cfe-commits
ericLemanissier added a comment. I have an segfault on all the source files of my project when I enable this check (it works ok when I disable this check). Sorry I don't have time to post a minimal source file producing the segfault. I will maybe tomorrow, or in two weeks. Repository: rL LL

[PATCH] D22196: Fix Bug "28480 - cppcoreguidelines-pro-bounds-array-to-pointer-decay handling __PRETTY_FUNCTION__"

2016-07-10 Thread Eric Lemanissier via cfe-commits
ericLemanissier created this revision. ericLemanissier added reviewers: alexfh, sbenza, bkramer, aaron.ballman. ericLemanissier added subscribers: ericLemanissier, cfe-commits. Herald added a subscriber: nemanjai. Ignore array to pointer decay for predefined macros. It is ok because there is no s

Re: [PATCH] D22196: Fix Bug "28480 - cppcoreguidelines-pro-bounds-array-to-pointer-decay handling __PRETTY_FUNCTION__"

2016-07-12 Thread Eric Lemanissier via cfe-commits
ericLemanissier abandoned this revision. ericLemanissier added a comment. Well, C++ Core Guidelines advises in this case to correct function receiving __PRETTY_FUNCTION__: the parameter type should not be const char* ( https://github.com/isocpp/CppCoreGuidelines/issues/640 ) I'm not sure if it s

Re: [PATCH] D22196: Fix Bug "28480 - cppcoreguidelines-pro-bounds-array-to-pointer-decay handling __PRETTY_FUNCTION__"

2016-07-14 Thread Eric Lemanissier via cfe-commits
ericLemanissier added a comment. After searching deeper into the C++ Core Guidelines, they are Ok with passing C strings to function without passing there size, but the types gsl::zstring and gsl::czstring have to be used : https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelin

Re: [PATCH] D22196: Fix Bug "28480 - cppcoreguidelines-pro-bounds-array-to-pointer-decay handling __PRETTY_FUNCTION__"

2016-07-14 Thread Eric Lemanissier via cfe-commits
ericLemanissier added a comment. having studied cpp core guidelines in more depth, I completely understand that the problem in the case I described is that the function receiving PRETTY_FUNCTION takes a const char* parameter, whereas it should take a czstring or szstring_span. As a consequence