From 044953b86672726acb1188a024dac5ce8b0cee7c Mon Sep 17 00:00:00 2001 From: "Yukihiro \"Matz\" Matsumoto" Date: Thu, 24 Jul 2025 15:51:17 +0900 Subject: [PATCH] mruby-string-ext: ensure newline before `else` keyword --- mrbgems/mruby-string-ext/src/string.c | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/mrbgems/mruby-string-ext/src/string.c b/mrbgems/mruby-string-ext/src/string.c index de12de131..28dd44e4c 100644 --- a/mrbgems/mruby-string-ext/src/string.c +++ b/mrbgems/mruby-string-ext/src/string.c @@ -1503,7 +1503,8 @@ str_lstrip_bang(mrb_state *mrb, mrb_value self) if (start < len) { memmove(ptr, ptr + start, len - start); RSTR_SET_LEN(s, len - start); - } else { + } + else { /* All whitespace - make empty */ RSTR_SET_LEN(s, 0); } @@ -1644,7 +1645,8 @@ str_chars_ary(mrb_state *mrb, mrb_value self) mrb_ary_push(mrb, result, char_str); p++; } - } else { + } + else { #ifdef MRB_UTF8_STRING /* UTF-8: handle multi-byte characters */ while (p < e) { @@ -1652,7 +1654,8 @@ str_chars_ary(mrb_state *mrb, mrb_value self) if (char_len == 0 || char_len > 4 || p + char_len > e) { /* Invalid UTF-8, treat as single byte */ char_len = 1; - } else { + } + else { /* Validate UTF-8 sequence */ mrb_bool valid = TRUE; if (char_len > 1) { @@ -1723,7 +1726,8 @@ str_ljust_core(mrb_state *mrb, mrb_value self) if (chars_needed >= pad_char_len) { mrb_str_cat_str(mrb, padding, padstr); chars_needed -= pad_char_len; - } else { + } + else { /* Need partial padding - use substr to get exact characters */ mrb_value partial = mrb_str_substr(mrb, padstr, 0, chars_needed); mrb_str_cat_str(mrb, padding, partial); @@ -1773,7 +1777,8 @@ str_rjust_core(mrb_state *mrb, mrb_value self) if (chars_needed >= pad_char_len) { mrb_str_cat_str(mrb, padding, padstr); chars_needed -= pad_char_len; - } else { + } + else { /* Need partial padding - use substr to get exact characters */ mrb_value partial = mrb_str_substr(mrb, padstr, 0, chars_needed); mrb_str_cat_str(mrb, padding, partial); @@ -1826,7 +1831,8 @@ str_center_core(mrb_state *mrb, mrb_value self) if (chars_needed >= pad_char_len) { mrb_str_cat_str(mrb, left_padding, padstr); chars_needed -= pad_char_len; - } else { + } + else { mrb_value partial = mrb_str_substr(mrb, padstr, 0, chars_needed); mrb_str_cat_str(mrb, left_padding, partial); chars_needed = 0; @@ -1840,7 +1846,8 @@ str_center_core(mrb_state *mrb, mrb_value self) if (chars_needed >= pad_char_len) { mrb_str_cat_str(mrb, right_padding, padstr); chars_needed -= pad_char_len; - } else { + } + else { mrb_value partial = mrb_str_substr(mrb, padstr, 0, chars_needed); mrb_str_cat_str(mrb, right_padding, partial); chars_needed = 0;