On Thu, 26 Jan 2017, Markus Heiser <markus.heiser@xxxxxxxxxxx> wrote: > Am 25.01.2017 um 21:59 schrieb Jani Nikula <jani.nikula@xxxxxxxxx>: > >>> But the problem I see here is, that the perl script generates a >>> reST output which I can't use. As an example we can take a look at >>> the man-page builder I shipped in the series. >> >> Sorry, I still don't understand *why* you can't use the same rst. Your >> explanation seems to relate to man pages, but man pages come >> *afterwards*, and are a separate improvement. I know you talk about lack >> of proper structure and all that, but *why* can it strictly not be used, >> if the *current* rst clearly can be used? > > "afterwards" is the word, that lets me slowly realize, that I have to > stop solving the world's problems with one patch. Now I guess how my > next patch series has to look like. Thanks! ... for being patient with > me. Indeed, we change the world, one small incremental patch at a time. ;) > Before I start, I want to hear your thoughts about the parsing > aspect ... > >>>> That said, perhaps having an elegant parser (perhaps based on a >>>> compiler plugin) is incompatible with the idea of making it a >>>> bug-for-bug drop-in replacement of the old one, and it's something >>>> we need to think about. > > Did you have any suggestions? The perfect is the enemy of the good... If we see that the current Perl parser just rewritten in Python really is an improvement, we should consider it. But as I wrote, there are still issues there, like performance, that we need to understand. I'll mostly defer to Jon on this. But before we plunge on with this, I would like to see at least some research into reusing existing parsers which I would expect are plentiful. We may end up deciding regexps are the way to go after all, but I'd like it to be based on a decision rather than a lack of one. And we might decide to look at this as a later improvement instead as well. I've looked at python-clang myself, but it's a huge dependency, and it's not trivial to cover all the things that the current one does with that. I'd dismiss that. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center -- 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
