Modify brackets ?

[icsl-Jeon]

los-keeper/los_keeper_ros2/src/los_server/los_server.cc

Lines 89 to 119 in da48a7d

	{ // Obstacle path
	visualization_.obstacle_path_vis = visualizer_.VisualizeObstaclePathArray(
	debug_info.obstacle_manager.structured_obstacle_poly_list);
	visualization_.obstacle_path_vis_publisher->publish(visualization_.obstacle_path_vis);
	}
	{ // Target raw path
	visualization_.target_raw_path_vis =
	visualizer_.VisualizeRawTargetPathArray(debug_info.target_manager.primitives_list);
	visualization_.target_raw_path_vis_publisher->publish(visualization_.target_raw_path_vis);
	}
	{ // Target safe path
	visualization_.target_safe_path_vis = visualizer_.VisualizeSafeTargetPathArray(
	debug_info.target_manager.primitives_list,
	debug_info.target_manager.primitive_safe_total_index);
	visualization_.target_safe_path_vis_publisher->publish(visualization_.target_safe_path_vis);
	}
	{ // Target best path
	visualization_.target_best_path_vis = visualizer_.VisualizeBestTargetPathArray(
	debug_info.target_manager.primitives_list, debug_info.target_manager.primitive_best_index);
	visualization_.target_best_path_vis_publisher->publish(visualization_.target_best_path_vis);
	}
	{ // Keeper raw path
	visualization_.keeper_raw_path_vis =
	visualizer_.VisualizeRawKeeperPathArray(debug_info.planning.primitives_list);
	visualization_.keeper_raw_path_vis_publisher->publish(visualization_.keeper_raw_path_vis);
	}
	{ // Keeper safe path
	visualization_.keeper_safe_path_vis = visualizer_.VisualizeSafeKeeperPathArray(
	debug_info.planning.primitives_list, debug_info.planning.safe_visibility_index);
	visualization_.keeper_safe_path_vis_publisher->publish(visualization_.keeper_safe_path_vis);
	}

If there were no reasons to use the indent-brackets, is it okay to remove them?

[YunwooLee94]

I've added comments to explain the code inside the bracket.
Do you think it's better to remove them?

[icsl-Jeon]

In our code, we usually have used bracket only when mutex lock is working (to decide exclusive region).
Its up to you! I will add more comments other than this one.

[YunwooLee94]

Okay For the sake of code coherence, I will remove the brackets, except when the mutex lock is in operation:)
Thx for your opinion.