Re: kernel-doc mishandles declarations split into lines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2017-06-06 at 10:59 -0300, Mauro Carvalho Chehab wrote:
> 
> A trivial "fix" would be to use just one line for the struct field :-
> )

Sure, we did this, but it makes checkpatch unhappy. We have a
relatively long struct name, a relatively long member name, and then
it's also an array so you have another constant that needs to fit ...
(I didn't puth the array part into the example, but without that it'd
actually fit on 80 cols)

> The logic that handle structs is at sub dump_struct() function at
> kernel-doc, with is called by dump_declaration().
> 
> I added some debug prints at kernel-doc...
> I guess the problem you're noticing is at process_proto_type():
> 
[...]
> Basically, that while(1) loop there seems to be misinterpreting the
> line with "very_long_member_name;"

Oh, that's possible - I may have been looking in the wrong place.

johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux