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