r181071 - [Doc parsing] Provide diagnostics for unknown documentation

jahanian fjahanian at apple.com
Fri May 3 17:10:07 PDT 2013


On May 3, 2013, at 5:07 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:

> On Sat, May 4, 2013 at 3:02 AM, jahanian <fjahanian at apple.com> wrote:
>> 
>> On May 3, 2013, at 4:57 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>> 
>>> On Sat, May 4, 2013 at 2:15 AM, Fariborz Jahanian <fjahanian at apple.com> wrote:
>>>> Author: fjahanian
>>>> Date: Fri May  3 18:15:20 2013
>>>> New Revision: 181071
>>>> 
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=181071&view=rev
>>>> Log:
>>>> [Doc parsing] Provide diagnostics for unknown documentation
>>>> commands. // rdar://12381408
>>> 
>>> I think this diagnostic will be extremely noisy (should it be off by
>>> default?).  We should definitely do typo-correction -- there's a list
>> 
>> I don't know. It caught two unknown in clang's tests. But, we don't want
>> to have too many new warnings where I come from. So, I will make it ignored
>> for now.
>> 
>>> of known commands available.
>> 
>> Yes, there will be a typo correction coming (it is part of above radar).
> 
> This is a weird suggestion (there are no other warnings like that),

I don't understand. Which suggestion is weird :)?
- Fariborz

> but anyway: we could emit this diagnostic by default only if we have a
> very good correction.
> 
> Dmitri
> 
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/




More information about the cfe-commits mailing list