r181487 - put noisy "unknown command tag name" warning
dgregor at apple.com
Thu May 9 08:57:19 PDT 2013
On May 9, 2013, at 8:53 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> On Thu, May 9, 2013 at 2:38 AM, Fariborz Jahanian <fjahanian at apple.com> wrote:
>> Author: fjahanian
>> Date: Wed May 8 18:38:56 2013
>> New Revision: 181487
>> URL: http://llvm.org/viewvc/llvm-project?rev=181487&view=rev
>> put noisy "unknown command tag name" warning
>> under -Wdocumentation-unknown-command and off by default.
>> patch by Dmitri Gribenko.
> Hi Fariborz, Doug,
> This turned out to be not good enough:
> We could fix all the warnings (doable, there are just a few headers
> that are included all over the place), but writing '\n' in a comment
> is not too uncommon, IMHO. So I think we should not warn about
> unknown single-character commands (or at least in cases where it is a
> known escape sequence).
> What do you think about this?
This seems like a reasonable heuristic.
> (My personal opinion is that selecting \ or @ for a command marker in
> Doxygen was a bad choice, but that ship has sailed long ago...)
We don't get to fix that now :(
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits