[llvm-branch-commits] [cfe-branch] r71418 - in /cfe/branches/Apple/Dib: lib/Analysis/CFRefCount.cpp test/Analysis/retain-release-gc-only.m

Bill Wendling wendling at apple.com
Mon May 11 10:46:31 PDT 2009


On May 10, 2009, at 8:43 PM, Mike Stump wrote:

> Author: mrs
> Date: Sun May 10 22:43:16 2009
> New Revision: 71418
>
> URL: http://llvm.org/viewvc/llvm-project?rev=71418&view=rev
> Log:
> Merge in 71397:
>
> Add special warning about returning a retained object where a GC'ed  
> object is expected.
>
> Modified:
>    cfe/branches/Apple/Dib/lib/Analysis/CFRefCount.cpp
>    cfe/branches/Apple/Dib/test/Analysis/retain-release-gc-only.m
>
> Modified: cfe/branches/Apple/Dib/lib/Analysis/CFRefCount.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/branches/Apple/Dib/lib/Analysis/CFRefCount.cpp?rev=71418&r1=71417&r2=71418&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- cfe/branches/Apple/Dib/lib/Analysis/CFRefCount.cpp (original)
> +++ cfe/branches/Apple/Dib/lib/Analysis/CFRefCount.cpp Sun May 10  
> 22:43:16 2009
> @@ -2517,9 +2517,10 @@
>   else if (RV->getKind() == RefVal::ErrorGCLeakReturned) {
>     ObjCMethodDecl& MD = cast<ObjCMethodDecl>(BRC.getCodeDecl());
>     os << " and returned from method '" <<  
> MD.getSelector().getAsString()
> -       << "' is potentially leaked when using garbage collection.   
> Callers"
> -          " of this method do not expect a +1 retain count since  
> the return"
> -          " type is an Objective-C object reference";
> +       << "' is potentially leaked when using garbage collection.   
> Callers "
> +          "of this method do not expect a returned object with a +1  
> retain "

Shouldn't this be "an object"?

-bw




More information about the llvm-branch-commits mailing list