On Thu, Jun 2, 2016 at 9:16 AM, Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> wrote: > On Thu, 02 Jun 2016, Daniel Vetter <daniel.vetter@xxxxxxxx> wrote: >> Further up in the state machinery we switch from STATE_NAME to >> STATE_DOCBLOCK when we match /$doc_block/. Which means this >> block of code here is entirely unreachable. >> >> Nuke it. > > kernel-doc currently supports multiple DOC: lines in a single kernel-doc > comment, and this breaks it. This: > > /** > * DOC: foo > * > * Foo > * > * DOC: bar > * > * Bar > */ > > currently turns into sections foo and bar, with this patch this becomes > one section "foo", with literal "DOC: bar" in the content as normal > text. > > I don't know if we actually have that kind of comments anywhere, but > since this change doesn't really help you all that much with the > follow-up, I'd just drop it. Fine with me, just wanted to drop the cargo-culting factor a bit. I'm too lazy to write a stateful script to check whether we have any place that uses the above ;-) If anyone can do this quickly, would be awesom. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- 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
