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

IJoypadEventDriven deprecated #1747

Closed
mebbaid opened this issue Jun 14, 2018 · 4 comments · Fixed by #1784
Closed

IJoypadEventDriven deprecated #1747

mebbaid opened this issue Jun 14, 2018 · 4 comments · Fixed by #1784
Assignees
Labels
Affects: YARP v3.0.0 This is a known issue affecting YARP v3.0.0 Component: Devices Component: Library - YARP_dev Fixed in: YARP v3.0.1 Issue Type: Bug Involves some intervention from a system administrator Resolution: Fixed

Comments

@mebbaid
Copy link
Contributor

mebbaid commented Jun 14, 2018

Describe the bug
When clean building yarp, and invoking IJoypadController from a walking module the following error apears:

In file included from /usr/local/src/robot/yarp/build/src/libYARP_conf/include/yarp/conf/api.h:13:0,
                 from /usr/local/src/robot/yarp/src/libYARP_OS/include/yarp/os/api.h:13,
                 from /usr/local/src/robot/yarp/src/libYARP_OS/include/yarp/os/LogStream.h:16,
                 from /usr/local/src/robot/element_capture-point-walking/code/cpp/Joypad_module/src/main.cpp:10:
/usr/local/src/robot/yarp/src/libYARP_dev/include/yarp/dev/IJoypadController.h:257:33: error: expected unqualified-id before ‘[’ token
     explicit IJoypadEventDriven(YARP_DEPRECATED_MSG("Use IJoypadEventDriven(double)") int rate);
                                 ^
/usr/local/src/robot/yarp/src/libYARP_dev/include/yarp/dev/IJoypadController.h:257:33: error: expected ‘)’ before ‘[’ token
Joypad_module/CMakeFiles/WalkingJoypadModule.dir/build.make:62: recipe for target 'Joypad_module/CMakeFiles/WalkingJoypadModule.dir/src/main.cpp.o' failed
make[2]: *** [Joypad_module/CMakeFiles/WalkingJoypadModule.dir/src/main.cpp.o] Error 1

However if i remove the YARP_DEPRECATED_MSG from the corresponding file and build yarp again, and then re-build my walking module, the error doesn't appear.

To Reproduce

  • You have to have the element_capture-point-walking module, precisely the virtualizer/CVirt Fork/branch. or any code that uses that class.
  • Build the source code and you will get the error.

Configuration (please complete the following information):

  • OS: ubuntu
  • yarp version: latest devel 3.0.0
  • compiler: gcc 5.4.0

@aerydna

@Nicogene Nicogene added Issue Type: Bug Involves some intervention from a system administrator Component: Library - YARP_dev Component: Devices Affects: YARP v3.0.0 This is a known issue affecting YARP v3.0.0 labels Jun 14, 2018
@Nicogene
Copy link
Member

Nicogene commented Jun 14, 2018

@drdanz the placement of YARP_DEPRECATED_MSG seems to give problems.

@traversaro
Copy link
Member

I guess to reproduce it is sufficient to compile any example that includes yarp/dev/IJoypadController.h using C++14 .

@drdanz
Copy link
Member

drdanz commented Jun 14, 2018

Probably we need yet another macro... something like YARP_DEPRECATED_CONSTRUCTOR?

@jeljaik
Copy link
Collaborator

jeljaik commented Jul 6, 2018

Same problem here for our project. We will just get rid of it manually. It doesn't appear when compiling with clang.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects: YARP v3.0.0 This is a known issue affecting YARP v3.0.0 Component: Devices Component: Library - YARP_dev Fixed in: YARP v3.0.1 Issue Type: Bug Involves some intervention from a system administrator Resolution: Fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants