<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On May 8, 2013, at 4:00 PM, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On Thu, May 9, 2013 at 1:54 AM, jahanian <<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>> wrote:<br><blockquote type="cite"><br>On May 8, 2013, at 3:52 PM, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:<br><br>On Sat, May 4, 2013 at 3:47 AM, Fariborz Jahanian <<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>><br>wrote:<br><br>Author: fjahanian<br>Date: Fri May 3 19:47:28 2013<br>New Revision: 181081<br><br>URL:<span class="Apple-converted-space"> </span><a href="http://llvm.org/viewvc/llvm-project?rev=181081&view=rev">http://llvm.org/viewvc/llvm-project?rev=181081&view=rev</a><br>Log:<br>[doc parsing]: Make warning about unknown command<br>tags off by default for now. Move diagnostic code<br>to DiagnosticCommentKinds.td. //<span class="Apple-converted-space"> </span><a href="rdar://12381408">rdar://12381408</a><br><br><br>I think this should be moved a separate group (for example, a separate<br>flag under DocumentationPedantic group). It causes way too much noise<br>even for LLVM and Clang.<br><br><a href="http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast/builds/2344/steps/build/logs/stdio">http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast/builds/2344/steps/build/logs/stdio</a><br><br><br>I agree. I was just looking at the log.<br></blockquote><br>What do you think about -Wdocumentation-unknown-command flag name? I<br>can not think of anything shorter...<br></div></blockquote><div><br></div>Fine by me.</div><div><br><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><blockquote type="cite">(There's also a crash somewhere in diagnostic printing, BTW…)<br><br><br>I just fixed the crash<br></blockquote><br>r181479? Erm, sort of. The root cause of the crash is still there.<br></div></blockquote></div><div><br></div>Well, we need the set to include the flag you mention about. Fariborz, once you've done the above, please gather the set of commits (which should be very, very small!) needed to make this warning non-default on the 3.3 branch. We don't want to pull typo correction into 3.3 to do it, though.<div><br><div><span class="Apple-tab-span" style="white-space:pre"> </span>- Doug</div><div><br></div></div></body></html>