Alp, you're not on reviews.llvm.org, so I'm cc-ing you here, as you may
want to look at this.


On Thu, May 22, 2014 at 9:23 PM, Alexander Kornienko <[email protected]>wrote:

> Hi rsmith,
>
> The limits on the number of fix-it hints and ranges attached to a
> diagnostic are arbitrary and don't apply universally to all users of the
> DiagnosticsEngine. The way the limits are enforced may lead to diagnostics
> generating invalid sets of fixes. I suggest removing the limits, which
> will also
> simplify the implementation.
>
> http://reviews.llvm.org/D3879
>
> Files:
>   .gitignore
>   include/clang/Basic/Diagnostic.h
>   include/clang/Basic/PartialDiagnostic.h
>   lib/Basic/Diagnostic.cpp
>   lib/Format/Format.cpp
>   lib/Frontend/SerializedDiagnosticPrinter.cpp
>   lib/Frontend/TextDiagnosticPrinter.cpp
>   lib/Lex/LiteralSupport.cpp
>   lib/Lex/PPMacroExpansion.cpp
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to