On Jul 21, 2008, at 3:09 PM, Ted Kremenek wrote:

> Author: kremenek
> Date: Mon Jul 21 17:09:15 2008
> New Revision: 53882
>
> URL: http://llvm.org/viewvc/llvm-project?rev=53882&view=rev
> Log:
> Add test case for nonnull attribute.
> Fix indexing bug.

Hey Ted,

Please use utostr in llvm/ADT/StringExtras.h instead of  
ostringstream.  Thanks!

-Chris

>     if (x < 1 || x > NumArgs) {
> +      std::ostringstream os;
> +      os << I.getArgNum();
>       S.Diag(Attr.getLoc(),  
> diag::err_attribute_argument_out_of_bounds,
> -             "nonnull", Ex->getSourceRange());
> +             "nonnull", os.str(), Ex->getSourceRange());
>       return;
>     }
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to