Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes doxygen issue #4457

Merged
merged 1 commit into from
Jan 9, 2019
Merged

Conversation

gf712
Copy link
Member

@gf712 gf712 commented Jan 9, 2019

Doxygen doesn't seem to like arguments packs and variadic templates and gives an error in these lines of shogun_doxygen.i:

// File: structshogun_1_1type__internal_1_1function__traits_3_01Ret_07F_1_1_5_08_07Args_8_8_8_08_01const_01_4.xml
%feature("docstring") shogun::type_internal::function_traits<
Ret(F::*)(Args...) const > " "; --> ***this line causes an issue****

and

%feature("docstring") shogun::Types< T1, Args... > " ";

This PR tells doxygen to ignore the corresponding C++ code.

doxygen doesn't seem to like variadic templates
@vigsterkr
Copy link
Member

i've killed this CI job... coz the windows build would have taken ages :) but it should be fine to merge. i'll merge it as soon as the #4456 is done

@vigsterkr vigsterkr merged commit d676c18 into shogun-toolbox:develop Jan 9, 2019
@gf712 gf712 deleted the type_case_doxygen_fix branch January 9, 2019 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants