eudoxia0 / docparser

Extract documentation from Common Lisp systems
41 stars 18 forks source link

Fix missing lambda-list of cffi functions defined without docstrings. #31

Closed avodonosov closed 2 years ago

avodonosov commented 2 years ago

If cffi function is defined without docstring, the cffi-functoin node created for it has nil lambda list. Due to typo in the parsing code.

A test is added. Without the fix in parsers.lisp the test fail; with the fix it passes. The rest of the test suite works as before.

As you see, the test is not relying of the parsing node to be at specific integer position in the index, as most of the testsuite does - that seems fragile (see #29). Instead the new tes just queries for the needed node.

eudoxia0 commented 2 years ago

Thanks!