r181487 - put noisy "unknown command tag name" warning
fjahanian at apple.com
Thu May 9 09:17:28 PDT 2013
Ouch, \t and \n are everywhere. Patch is coming.
On May 9, 2013, at 8:57 AM, Douglas Gregor <dgregor at apple.com> wrote:
> 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 :(
> - Doug
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits