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

changes between 1.12.13 and 1.13.7 for backporting #1490

Merged
merged 13 commits into from
Aug 23, 2018

Conversation

dirk-thomas
Copy link
Member

The following list of changes has been integrated into ros_comm 1.13.7 (Lunar) since the last Kinetic release (1.12.13).

Commits already in kinetic-devel:

Backported: (these changes are part of this PR)

Not backported: (present on the lunar-devel branch and/or part of the backport to Lunar #1477)

@ros/ros_team @ros/ros_comm-maintainers RFC.

Enrique Fernández Perdomo and others added 13 commits August 20, 2018 14:12
This allows to use the operator<< also with const references to
XmlRpcValue objects, not only non-const ones.
* Skip empty lines

* Only check 'Header ' is at the beginning

This allows to support sensor_msgs/Image, which has a comment after.
Also note we only need to check for the type, not the name of the field.

* Break while loop after first field.
* Add substitution when loading yaml files

like in #1051

* Add tests
* Template std::max instead of casting explicitly.
* Change to force a rebuild of the pollset on flag changes

This change fixes issues seen in #1357, resulting from #1281
* Make a copy of the topic list

get_topics() returns a set, which is iterated by the Log constructor.
Make a copy of the set (atomic in cpython!) so nobody adds or removes
something while the Log constructor iterates over it

* Revert "Make a copy of the topic list", Fix

This reverts commit 0e019e0.
Move the fix to the _TopicManager so anybody else calling get_topics doesn't get burned by the same issue as _rosout
…1450)

* Rename getLastestTime to getLatestTime

Per directions [here](#1449 (comment)).

* Test getLatestTime instead of getLastestTime

per #1449 (comment)
@dirk-thomas dirk-thomas self-assigned this Aug 20, 2018
@dirk-thomas
Copy link
Member Author

@ros-pull-request-builder retest this please

@dirk-thomas dirk-thomas merged commit 09880c7 into kinetic-devel Aug 23, 2018
@dirk-thomas dirk-thomas deleted the kinetic-backports branch August 23, 2018 18:28
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.

10 participants