r207392 - libclang: split out the documentation comment API

Tobias Grosser tobias at grosser.es
Mon Apr 28 01:27:28 PDT 2014


On 28/04/2014 10:09, Alp Toker wrote:
>
> On 28/04/2014 08:37, Dmitri Gribenko wrote:
>> On Mon, Apr 28, 2014 at 8:16 AM, Alp Toker <alp at nuanti.com> wrote:
>>
>>> Most of all I'm certain that ~20,000 LoC can be reduced for what
>>> boils down
>>> to parsing a few comments and checking that param commands match the
>>> user's
>>> function parameters.
>> Given that I am the primary author of this code, I am skeptical of
>> such claims.  Doxygen is complicated, non-trivial to parse, and
>> non-uniform in its syntax and semantic rules.
>
> I really appreciate that and I want to help get your work in the hands
> of more users. The goal is most certainly not to trivialise the work,
> but to see if there are ways to make it move faster :-)

Hi Dimitry,

I would like to jump in and backup Alp here. Having seen the commits 
that add HTML verification to clang I was also very surprised. I am sure 
you have good reasons to do so, but as implementing full HTML parsing 
and verification in clang is at least surprising, I would personally be 
interested in a cfe-dev@ email that explains your motivation and 
reasoning, other possible options (e.g. external libraries) and why you 
decided to go for the approach you take.

Thanks,
Tobias



More information about the cfe-commits mailing list