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