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

Fixed documentation warnings #714

Merged
merged 1 commit into from Oct 9, 2014

Conversation

Projects
None yet
3 participants
@mantognini
Member

mantognini commented Oct 7, 2014

Such as:

  • WindowImpl.hpp:94:16: warning: parameter 'threshold:' not found in the function declaration
  • SFContext.hpp:120:16: warning: parameter 'enabled:' not found in the function declaration

Should not be a big deal. :)

Fixed documentation warnings
 - WindowImpl.hpp:94:16: warning: parameter 'threshold:' not found in the function declaration
 - SFContext.hpp:120:16: warning: parameter 'enabled:' not found in the function declaration

@mantognini mantognini self-assigned this Oct 7, 2014

@mantognini mantognini added this to the 2.2 milestone Oct 7, 2014

@mantognini mantognini added the bug label Oct 7, 2014

@TankOs

This comment has been minimized.

Show comment
Hide comment
@TankOs

TankOs Oct 7, 2014

Member

👍

Member

TankOs commented Oct 7, 2014

👍

@eXpl0it3r

This comment has been minimized.

Show comment
Hide comment
@eXpl0it3r

eXpl0it3r Oct 8, 2014

Member

There are a lot more doxygen warnings being generated, but I guess we can deal with them at a later point.

This branch will otherwise be merged soon.

Member

eXpl0it3r commented Oct 8, 2014

There are a lot more doxygen warnings being generated, but I guess we can deal with them at a later point.

This branch will otherwise be merged soon.

@mantognini

This comment has been minimized.

Show comment
Hide comment
@mantognini

mantognini Oct 8, 2014

Member

Those warnings are emitted by clang. In fact I haven't looked at the doxygen output for a loooong time but I guess you're right.

Member

mantognini commented Oct 8, 2014

Those warnings are emitted by clang. In fact I haven't looked at the doxygen output for a loooong time but I guess you're right.

@eXpl0it3r eXpl0it3r merged commit b2ba230 into master Oct 9, 2014

@eXpl0it3r eXpl0it3r deleted the bugfix/warnings branch Oct 9, 2014

@eXpl0it3r eXpl0it3r added the s:accepted label Oct 9, 2014

@mantognini mantognini removed their assignment Apr 30, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment