You should start your diagnostic text with a lowercase letter. I'm also curious what you're diagnosing here! (Missing import of macros, maybe?)<br><br><div>On Fri Jan 31 2014 at 12:47:21 PM, Thompson, John <<a href="mailto:John_Thompson@playstation.sony.com">John_Thompson@playstation.sony.com</a>> wrote:</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Thank you both, Zach and Chad!<u></u><u></u></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">-John<u></u><u></u></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> Zach Davis [mailto:<a href="mailto:zdavkeos@gmail.com" target="_blank">zdavkeos@gmail.com</a>]
<br>
<b>Sent:</b> Friday, January 31, 2014 11:55 AM<br>
<b>To:</b> Thompson, John<br>
<b>Cc:</b> <a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<b>Subject:</b> Re: [cfe-dev] How to add a -W flag?<u></u><u></u></span></p></div></div><div lang="EN-US" link="blue" vlink="purple"><div>
<p><u></u> <u></u></p>
<div>
<p style="margin-bottom:12.0pt">I believe you need to add your warning to a group like so:<br>
<br>
  def warn_need_module_import : Warning<<br>
    "Use of identifier '%0' requires import/inclusion of the module '%1'">;<br>
<br>
becomes --><br>
<br>
  def warn_need_module_import : Warning<<br>
    "Use of identifier '%0' requires import/inclusion of the module '%1'">,<br>
    InGroup<SomeGroup>;<br>
<br>
where "SomeGroup" is taken from DiagnosticGroups.td .<br>
<br>
You can add you own group as well, the format is like:<br>
<br>
  def SomeGroup : DiagGroup<"some-group">;<br>
<br>
and then you should be able to use -Wsome-group<br>
<br>
<u></u><u></u></p>
</div>
<div>
<p style="margin-bottom:12.0pt"><u></u> <u></u></p>
<div>
<p>On Fri, Jan 31, 2014 at 1:29 PM, Thompson, John <<a href="mailto:John_Thompson@playstation.sony.com" target="_blank">John_Thompson@playstation.sony.com</a>> wrote:<u></u><u></u></p>
<div>
<div>
<p>I’m adding a warning to the end of the “Modules Issues” category in DiagnosticSemaKinds.td:<u></u><u></u></p>
<p> <u></u><u></u></p>
<p style="text-autospace:none">
<span style="font-size:9.5pt;font-family:Consolas;background:white">def warn_need_module_import : Warning<</span><u></u><u></u></p>
<p style="text-autospace:none">
<span style="font-size:9.5pt;font-family:Consolas;background:white">  "Use of identifier '%0' requires import/inclusion of the module '%1'">;</span><u></u><u></u></p>
<p> <u></u><u></u></p>
<p>This makes test/Misc/warning-flags.c fail.<u></u><u></u></p>
<p> <u></u><u></u></p>
<p>A comment in that test file says:<u></u><u></u></p>
<p> <u></u><u></u></p>
<p>    If you add a new warning without a flag, this test will fail.  To fix<u></u><u></u></p>
<p>    this test, simply add a warning group to that warning.<u></u><u></u></p>
<p> <u></u><u></u></p>
<p>I don’t know what that means.  How do I add a warning group to the warning?<u></u><u></u></p>
<p> <u></u><u></u></p>
<p>Thanks.<u></u><u></u></p>
<p><span style="color:#888888"> <u></u><u></u></span></p>
<p><span style="color:#888888">-John<u></u><u></u></span></p>
<p><span style="color:#888888"> <u></u><u></u></span></p>
<p><span style="color:#888888"> <u></u><u></u></span></p>
</div>
</div>
<p style="margin-bottom:12.0pt"><br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><u></u><u></u></p>
</div>
<p><u></u> <u></u></p>
</div>
</div></div></blockquote>