fix new linter warnings as of flake8-comprehensions 3.1.0 (ros2#94)

Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
esteve · Nov 15, 2019 · 25a4a39 · 25a4a39
1 parent c49183c
commit 25a4a39
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/launch_ros/launch_ros/utilities/normalize_parameters.py b/launch_ros/launch_ros/utilities/normalize_parameters.py
@@ -135,7 +135,7 @@ def normalize_parameter_dict(
         if _prefix:
             # Prefix name if there is a recursive dictionary
             # weird looking logic to combine into one list to appease mypy
-            combined = [e for e in _prefix]
+            combined = list(_prefix)
             combined.append(TextSubstitution(text='.'))
             combined.extend(name)
             name = combined

diff --git a/launch_testing_ros/launch_testing_ros/tools/output.py b/launch_testing_ros/launch_testing_ros/tools/output.py
@@ -29,7 +29,7 @@ def get_rmw_output_filter(rmw_implementation, filter_type):
 
     # Treat each line of the resource as an independent filter.
     rmw_output_filter, _ = ament_index_python.get_resource(resource_name, rmw_implementation)
-    return [l for l in rmw_output_filter.splitlines()]
+    return rmw_output_filter.splitlines()
 
 
 def basic_output_filter(