Skip to content

[Aggregator] Fixes C++ Warnings (pluginlib)#82

Merged
trainman419 merged 1 commit intoros:indigo-develfrom
moriarty:fix-pluginlib-deprecated-headers
May 2, 2018
Merged

[Aggregator] Fixes C++ Warnings (pluginlib)#82
trainman419 merged 1 commit intoros:indigo-develfrom
moriarty:fix-pluginlib-deprecated-headers

Conversation

@moriarty
Copy link
Copy Markdown
Contributor

This fixes the following warnings:

warning: Including header <pluginlib/class_list_macros.h>
 is deprecated,include <pluginlib/class_list_macros.hpp> instead. [-Wcpp]

warning: Including header <pluginlib/class_loader.h>
 is deprecated, include <pluginlib/class_loader.hpp> instead. [-Wcpp]

The .hpp files have been backported to indigo

But just a few hours ago they've reverted the deprecation warnings (for now)
See:
ros/pluginlib/pull/70 ( ros/pluginlib#70 (comment))
ros/pluginlib/pull/115

This fixes the following warnings:

warning: Including header <pluginlib/class_list_macros.h>
 is deprecated,include <pluginlib/class_list_macros.hpp> instead. [-Wcpp]

warning: Including header <pluginlib/class_loader.h>
 is deprecated, include <pluginlib/class_loader.hpp> instead. [-Wcpp]

The .hpp files have been backported to indigo
@trainman419 trainman419 merged commit 76bfbf1 into ros:indigo-devel May 2, 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

Successfully merging this pull request may close these issues.

2 participants