[cfe-dev] [analyzer] introducing exceptions checkers

Kirill Bobyrev via cfe-dev cfe-dev at lists.llvm.org
Sun Sep 6 13:46:29 PDT 2015


Hi, Gabor!

Yes, I totally missed it. Thanks for pointing it out! I'll fix that 
soon. As well as adding the third checker 
(/exceptions.ThrownTypeDiffersSpec/) into the "master" one.

Many thanks,
Kirill

On 9/6/2015 11:42 PM, Gábor Horváth wrote:
> Hi!
>
> Thank you for working on the static analyzer! I have one high level 
> comment though. The noexcept or throw() does not mean that a function 
> can not contain a throw expression. It means that an exception can not 
> leak out from that function. So it is perfectly valid to throw an 
> exception and catch it (without rethrowing it) within a noexcept 
> function. This is something that you might want to consider (in a 
> future version).
>
> Regards,
> Gabor
>
> On 6 September 2015 at 22:08, Kirill Bobyrev via cfe-dev 
> <cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>> wrote:
>
>     I've done the initial version of exceptions checkers from the
>     potential checkers list
>     <http://clang-analyzer.llvm.org/potential_checkers.html>. I
>     actually found /exceptions.ThrowSpecButNotThrow/ and
>     /exceptions.NoThrowSpecButThrows/ very similar and decided to put
>     them together into /exceptions.ExceptionSpec/. It checks for
>     "throw calls" in functions specified as noexcept and functions,
>     which do not have "throw calls" but should have such calls inside
>     them by declaration.
>
>     The first version of checker is attached. I wonder if it's good
>     enough and I can push it for the review.
>
>     Feedback and suggestions highly appreciated!
>
>     --
>     Kirill Bobyrev
>     https://github.com/omtcyf0
>     noe
>     _______________________________________________
>     cfe-dev mailing list
>     cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>
>     http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20150906/6feb304b/attachment.html>


More information about the cfe-dev mailing list