From f80192e8acdafb1be16026b8fefc8f79ec716896 Mon Sep 17 00:00:00 2001 From: "Yukihiro \"Matz\" Matsumoto" Date: Wed, 24 May 2023 17:59:32 +0900 Subject: [PATCH] mruby-struct/struct.c (struct_s_member): avoid `break` Direct `return` is more readable, IMO. --- mrbgems/mruby-struct/src/struct.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/mrbgems/mruby-struct/src/struct.c b/mrbgems/mruby-struct/src/struct.c index 51c2c0eba..11f4bbb78 100644 --- a/mrbgems/mruby-struct/src/struct.c +++ b/mrbgems/mruby-struct/src/struct.c @@ -45,14 +45,13 @@ struct_s_members(mrb_state *mrb, struct RClass *c) if (!mrb_array_p(members)) { struct_corrupted(mrb); } - break; + return members; } c = c->super; if (c == sclass || c == 0) { mrb_raise(mrb, E_TYPE_ERROR, "uninitialized struct"); } } - return members; } static mrb_value