fix that if no parameter is updated, the user callback is not called

[fabianhirmann]

This can be the case if one node has multiple param listeners in multiple namespaces and then the add_on_set_parameters_callback is called for all the parameter changes and not just for the relevant ones.

[fabianhirmann]

After the pull request is merged, please backport the change to at least ROS 2 Humble as well.

[christophfroehlich]

Description sounds reasonable to be fixed, but can you please add tests for that?

[fabianhirmann]

Description sounds reasonable to be fixed, but can you please add tests for that?

Thank you for your quick response! I also added now a test in example/test/example_test_gtest.cpp because as far as I saw at the other test cases there is no testing of the behavior of the produced ParamListener. The newly created test passes with my changes and fails when tested without my changes.

By that I also noticed that I missed to update the generator for update_runtime_parameter but this is now also fixed (all tests should pass now (at least this is the case when testing the changes on ROS Humble)).

[christophfroehlich]

Thanks for the test, can you add one for the python module as well please?

[fabianhirmann]

Thanks for the test, can you add one for the python module as well please?

Thank you for the catch! I also added now a test for the python module.