Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

Added rosbag::Bag::isOpen#1190

Merged
dirk-thomas merged 1 commit intoros:lunar-develfrom
racko:added_bag_isOpen
Oct 20, 2017
Merged

Added rosbag::Bag::isOpen#1190
dirk-thomas merged 1 commit intoros:lunar-develfrom
racko:added_bag_isOpen

Conversation

@racko
Copy link
Copy Markdown
Contributor

@racko racko commented Oct 14, 2017

No description provided.


void swap(Bag&);

bool isOpen() const { return file_.isOpen(); }
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please keep the implementation of this method in the cpp file even though it is short.

@racko
Copy link
Copy Markdown
Contributor Author

racko commented Oct 17, 2017

Updated

@dirk-thomas
Copy link
Copy Markdown
Member

dirk-thomas commented Oct 20, 2017

The patch looks good. Since it changes ABI it might be better to target the next ROS distro. Nvm, the new method is non-virtual and should therefore not break ABI.

@dirk-thomas
Copy link
Copy Markdown
Member

@ros-pull-request-builder retest this please

@dirk-thomas dirk-thomas merged commit 293351f into ros:lunar-devel Oct 20, 2017
artivis pushed a commit to artivis/ros_comm that referenced this pull request Aug 9, 2019
dirk-thomas pushed a commit that referenced this pull request Feb 10, 2020
Co-authored-by: Tim Rakowski <tim.rakowski@googlemail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants