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

changes between 1.12.13 and 1.13.7 for backporting#1490

Merged
dirk-thomas merged 13 commits intokinetic-develfrom
kinetic-backports
Aug 23, 2018
Merged

changes between 1.12.13 and 1.13.7 for backporting#1490
dirk-thomas merged 13 commits intokinetic-develfrom
kinetic-backports

Conversation

@dirk-thomas
Copy link
Copy Markdown
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
Copy Markdown
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 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.

10 participants