Hello,<br><br>This is the first patch-set to better document clang diagnostics.<br><br>> I have added two fields into the Diagnostic (.td) class: Brief and Explanation; and modified the subsequent generation (and exploitation) of the table<br>
> I have added two getters to DiagnosticIDs to get the "BriefExplanation" and "FullExplanation"<br><br>> There is also a few corrections of comments that were (it seems) out of sync<br><br>Note: there is one (little) patch for LLVM, because the TableGen backend is in llvm repository... and the two patches are intrisically tied (obviously)<br>
<br>Note 2: is this normally tested or do we assume that is compilation a test in itself ?<br><br>Unless we want the same Brief / Explanation for Groups too, this normally implement the "libclang" part, so next are the HTML pages generation and the Help menu modification.<br>
<br><br>
For those tasks, the distinction of where the diagnostic comes from 
probably does not makes sense for the user, so I would expect to have to
 group the warnings using the categories and subgroups rather than AST/Sema/Parse. I suppose that 
we'll want to expose both warnings and extension warnings here.<br><br>
There is one catch though: the individual warnings are not named. I was
 wondering if I should create a name (#ENUM in the macro ? The internal ID ?) or if the Brief would be enough and I'd just 
list all the briefs/explanations underneath the subgroup they belong to.<br><br><br>With regard to the HTML generation: I was thinking about using a Python script and invoke it as part of the build step. This would require, I guess, that I update the Python Bindings (if any ?) to extract the information from libclang. I'd appreciate pointers to the documentation (if any).<br>
<br><br>Comments / Advices are welcome,<br>Matthieu.<br><br><div class="gmail_quote">2011/3/8 Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On Sun, 6 Mar 2011 14:05:14 +0100, Matthieu Monrocq said:<br>
<br>
>Note: regarding the activation of "everything", what of a "-Wextra" that<br>
>activates everything "-Wall" did not, like gcc ?<br>
<br>
</div>For the record, gcc does not do that.  -Wextra is more warnings than -<br>
Wall, but it is not everything.<br>
<font color="#888888"><br>
--<br>
____________________________________________________________<br>
Sean McBride, B. Eng                 <a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a><br>
Rogue Research                        <a href="http://www.rogue-research.com" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer              Montréal, Québec, Canada<br>
<br>
<br>
</font></blockquote></div><br>