<div class="gmail_quote">On Wed, Mar 7, 2012 at 11:35 AM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com">kremenek@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think I only have a minor nit:<br>
<br>
+  "the '%0' %1 is not supported by ISO C">, InGroup<FormatNonStandard>, DefaultIgnore;<br>
<br>
It seems weird to start a diagnostic with "the".  Why include it?  Dropping it changes:<br>
<br>
  the 'q' length modifier is not supported by ISO C<br>
<br>
to<br>
<br>
  'q' length modifier is not supported by ISO C<br>
<br>
The second seems just fine for a warning.<br></blockquote><div><br></div><div>I have another issue with this diagnostic: it's substituting %1 for either "conversion specifier" or "length modifier". That's a violation of the rules for diagnostics since it's not translatable. While you're here, can you change this to a %select?</div>
<div><br></div><div>Thanks!</div><div>Richard</div></div>