code style only: wrap after open parenthesis if not in one line

[dirk-thomas]

Style update to match the ROS 2 development guide and pass with the updated linter configuration from ament/ament_lint#210.

[wjwwood]

This is another instance where I think the rule is too aggressive. I think this is what should have been produced:

    visualizer_app_->loadConfig(QDir::toNativeSeparators(
      QString::fromStdString(
        package_share_directory + "/config/visual_tests_test_image_config.rviz")));

Actually this is probably best:

    visualizer_app_->loadConfig(QDir::toNativeSeparators(QString::fromStdString(
      package_share_directory + "/config/visual_tests_test_image_config.rviz")));

but given the original code, the first is reasonable for the linter to produce.

Basically it should be greedy, trying to get as many ( on the first line so long as the matching ) are not followed by additional arguments. I would, again, guess that the linter cannot handle such logic. So I guess it's more of a point in favor of maybe leaving this setting unset rather than deal with the "false positives" or "over matching" issues.