zaks.anna added a comment.

Looks like Sema uses "null" not only when referring to literals (see below). 
Also, if we were referring to literals, we would use single quotes, no?

I suggest keeping as is for consistency with the wording that uses nil. I do 
not see much difference between the two cases..

  TestObject * _Nonnull returnsNilObjCInstanceDirectly() {
     // The first warning is from Sema. The second is from the static analyzer.
     return nil; // expected-warning {{null returned from function that 
requires a non-null return value}}
                 // expected-warning@-1 {{nil returned from a function that is 
expected to return a non-null value}}
  }




https://reviews.llvm.org/D27600



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

Reply via email to