On Mon, 17 Oct 2016, Stephan Mueller <smuel...@chronox.de> wrote: > Am Montag, 17. Oktober 2016, 14:04:14 CEST schrieb Jani Nikula: > > Hi Jani, > >> The directive parameter is plural functions for a reason - you can >> specify multiple functions in the same directive. Splitting this to >> multiple directives causes the header file to be parsed again for each >> directive. >> >> IMO this can be fixed in a follow-up patch. Same for other patches in >> this series. > > Thank you very much for the hint. I followed the path what the DocBook > to Sphinx converter generated. I will change it in my patchset, but > may I suggest that the converter tool (tmplcvt) should be fixed, too?
I don't think it's worth the trouble. Now it's a straightforward docproc directive to Sphinx kernel-doc directive extension conversion, with one-to-one mapping. It would be quite a bit more complicated to gather all of the consecutive directives together, in a rather quickly hacked up tool which we'll throw away once all DocBooks have been converted over. And as I said, I think you can fix this up afterwards too. It's not broken, it's just a bit slower. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html