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

Functions with return type not returning anything #271

Closed
jlack1987 opened this issue May 2, 2017 · 3 comments
Closed

Functions with return type not returning anything #271

jlack1987 opened this issue May 2, 2017 · 3 comments

Comments

@jlack1987
Copy link
Contributor

Working on the current release of kinetic-devel it looks like there are some compiler warnings generated for a few methods in InterfaceManager that have a return type but don't have a return statement. The offenders are here, here, and here.

Haven't looked into what these methods actually do, so it might technically not be a problem, but since my group develops with -Werror on by default it's a pain to have to now go manually ignore it with -Wno-return-type. Also seems like kinda bad c++ foo. Thanks!

@bmagyar
Copy link
Member

bmagyar commented May 3, 2017

Hi Jordan,
Long time, no see ;)

That bit of metaprogramming arrived with combined_robot_hw and I am not fully familiar with it.
Maybe @toliver could help us out on this one?

@davetcoleman
Copy link
Member

+1 to fixing these warnings, i am experiencing them too on Lunar

@mathias-luedtke
Copy link
Contributor

should be fixed in #285 ;)

@bmagyar bmagyar closed this as completed Feb 1, 2018
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

No branches or pull requests

4 participants