Melvin Vermeeren

Results 116 comments of Melvin Vermeeren

@mbruno-xmos Hi, I checked and this was last changed in 2014 by 4380bf06 back when @michaeljones was actively developing Breathe (though that may change again, see #546). Patches are very...

Is this still a problem with the latest Sphinx/Breathe versions? I'm in the process of properly tagging issues and closing obsolete ones.

Assuming fixed (either here or upstream) for now, comment if not the case.

@YannickJadoul Can you post the RST in question? You should be able to make it work by specifying the arguments explicitly, the checks for this have gotten more strict in...

Just to make sure: there is only one `.. doxygenfunction:: augpy::check_contiguous(CudaTensor *)` in all of the RST docs? It might be that only the function name is used to identify...

Sphinx docs very much indicate this is fully supported in the C++ domain, see http://www.sphinx-doc.org/en/master/usage/restructuredtext/domains.html#overloaded-member-functions. Though the `Duplicate declaration` warning is emitted in Sphinx at https://github.com/sphinx-doc/sphinx/blob/3.x/sphinx/domains/cpp.py#L6556. I suspect either Breathe...

Released fix in v4.18.1, should be resolved now. @jfolz Could you test with the new release?

Thanks for the reply, will reopen this for now as the edge cases aren't fully covered yet it seems. Good to hear most instances of the problem are resolved now...

@jfolz The `Unable to resolve multiple matches for function` should be a lot better with #606, released in Breathe 4.25.0. However the `Duplicate declaration` warning likely still exists as before,...

The PR #605 mentioned this issue, and #606 provided a more proper fix for parsing function signatures. I am not entirely sure how or if this issue actually relates to...