Skip to content

Commit

Permalink
Merge pull request #24 in SDK/quanergy_client_ros from bugfix/SDK-126…
Browse files Browse the repository at this point in the history
…-userostime-is-not-being-used-correctly-by-the-publisher to develop

* commit '78d5a02b9f734bb164faab4b5c2a0fd68fe764fc':
  Update: bump patch version
  Fix: call other constructor through initalizer list to properly initialize useRosTime
  • Loading branch information
vaw committed Nov 29, 2018
2 parents a520762 + 78d5a02 commit 40c70a5
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
8 changes: 5 additions & 3 deletions include/quanergy_client_ros/simple_publisher.h
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,12 @@ class SimplePublisher
SimplePublisher(
const std::string& topic = "unnamed",
bool useRosTime = false)
: topic_(topic) , useRosTime_(useRosTime) {}
SimplePublisher(bool useRosTime)
: topic_(topic) , useRosTime_(useRosTime)
{
}

SimplePublisher(bool useRosTime) : SimplePublisher("unnamed", useRosTime)
{
SimplePublisher("unnamed", useRosTime);
}

void slot(const CloudConstPtr& cloud)
Expand Down
2 changes: 1 addition & 1 deletion package.xml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<?xml version="1.0"?>
<package>
<name>quanergy_client_ros</name>
<version>2.3.0</version>
<version>2.3.1</version>
<description>The quanergy_client_ros package provides a ROS driver for Quanergy sensors</description>
<maintainer email="[email protected]">Ross Taylor</maintainer>
<license>Copyright 2014- Quanergy Systems Inc.</license>
Expand Down

0 comments on commit 40c70a5

Please sign in to comment.