https://github.com/NagyDonat commented:

Thanks for this improvement, I'm really happy to see this! (I added some inline 
comments, but they are all minor and tangential.)   

Right now all the `nullability.*` checkers are marked as (ObjC) in [the 
documentation](https://clang.llvm.org/docs/analyzer/checkers.html#nullability), 
because (as far as I know) previously they were only supporting the Objective-C 
style `_Nullable` and `_Nonnull` annotations.

However, as your change shows, the same "backend" can be used to handle 
different kinds of nullability annotations, which would make these checkers 
available for more users.

Please update the documentation of the affected checkers (in 
`clang/docs/analyzer/checkers.rst`) to mark that they are no longer limited to 
Objective-C.



https://github.com/llvm/llvm-project/pull/106048
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to