[cfe-commits] Null init suggestion

David Blaikie dblaikie at gmail.com
Fri Aug 26 18:39:23 PDT 2011


>
> Side note: Do we really need that first note if we're providing the second
> note? I realize in some cases the first note is all we'll get, because
> there's no initialization we can guess to provide. But in the cases where
> there is an init suggestion, could we just provide that instead of the
> "variable '%s' is declared here"?
>
>
> Good point. It seems that we don't need to provide the first note if we
> know we're providing the second. Looping in Ted to get his opinion.
>

Well - here's a patch you can use if you decide it's the right way to go.
(I'm only seeing one clang test failure in something
unrelated: Modules/lookup.cpp)

And now, PAX.
- David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110826/bd773d90/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: null_init_suggestion.patch
Type: application/octet-stream
Size: 716 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110826/bd773d90/attachment.obj>


More information about the cfe-commits mailing list