ahatanak added a subscriber: cfe-commits.
ahatanak added a comment.

+cfe-commits

If this patch is applied, does clang issue a warning if a method marked 
"nonnull" returns a null value? I see a warning is issued for conditional 
expressions in the test case you've added, but I don't see a test case for a 
function returning just nil or 0.

I was wondering whether the change made in this patch contradicts what's stated 
in r240146's commit log:

"Note that we don't warn about nil returns from Objective-C methods,

  because it's common for Objective-C methods to mimic the nil-swallowing
  behavior of the receiver by checking ostensibly non-null parameters
  and returning nil from otherwise non-null methods in that
  case."


https://reviews.llvm.org/D23236



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to