From 49595c230a55a90db8544845ca157127f6246afe Mon Sep 17 00:00:00 2001 From: Dan Rose Date: Wed, 17 Jul 2019 13:48:48 -0500 Subject: [PATCH] Set indent_paren_open_brace=true Since we indent the arguments after a newline, this prevents a visual dedent when using a braced argument: BAD: ``` ASSERT_THROW( { auto client = node->create_client( "invalid_service?" ); }, rclcpp::exceptions::InvalidServiceNameError ); ``` GOOD: ``` ASSERT_THROW( { auto client = node->create_client( "invalid_service?" ); }, rclcpp::exceptions::InvalidServiceNameError ); ``` Signed-off-by: Dan Rose --- .../ament_uncrustify/configuration/ament_code_style.cfg | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ament_uncrustify/ament_uncrustify/configuration/ament_code_style.cfg b/ament_uncrustify/ament_uncrustify/configuration/ament_code_style.cfg index d030ece7..feac0843 100644 --- a/ament_uncrustify/ament_uncrustify/configuration/ament_code_style.cfg +++ b/ament_uncrustify/ament_uncrustify/configuration/ament_code_style.cfg @@ -769,7 +769,7 @@ indent_braces_no_struct = false # false/true indent_brace_parent = false # false/true # Indent based on the paren open instead of the brace open in '({\n', default is to indent by brace. -indent_paren_open_brace = false # false/true +indent_paren_open_brace = true # false/true # indent a C# delegate by another level, default is to not indent by another level. indent_cs_delegate_brace = false # false/true