Exception in parser about marginalia HOT 6 CLOSED

I've found out the cause of the problem and I'm working on a fix, should be available soon.

from marginalia.

The fix has been commited:

a92588f

The parser was reading comments has if they where forms, so an uneven number of comment in a map would throw an exception. I've changed the way the custom reader extract comments and made it append them to a vector placed inside an atom and returns the reader itself like the default CommentReader. Thus, that should prevent any other similar bugs.

The parser has been improved at the same time to handle nested comments properly, so I've started a discussion on what to do with them:

https://github.com/fogus/marginalia/issues/issue/15

from marginalia.

Still seems to fail for me with the same code. A different exception though:

Caused by: java.lang.IndexOutOfBoundsException
at clojure.lang.RT.nthFrom(RT.java:753)
at clojure.lang.RT.nth(RT.java:722)
at marginalia.parser$eval154$fn__155.invoke(parser.clj:101)
at clojure.lang.MultiFn.invoke(MultiFn.java:171)
at marginalia.parser$extract_docstring.invoke(parser.clj:138)
at marginalia.parser$arrange_in_sections.invoke(parser.clj:199)
at marginalia.parser$parse.invoke(parser.clj:218)
at marginalia.parser$parse_file.invoke(parser.clj:224)
at marginalia.core$path_to_doc.invoke(core.clj:125)
at clojure.core$map$fn__3695.invoke(core.clj:2096)
at clojure.lang.LazySeq.sval(LazySeq.java:42)

from marginalia.

OK, totally different bug, a very stupid mistake! It should be much easier to fix tough.

from marginalia.

Here you go Hugo!

30aec7c

I should have kept the original sample! I'll let you close this issue that time.

from marginalia.

Thanks for the swift fixes Nicolas. All well now.

from marginalia.