[Libguestfs] [libnbd PATCH 6/6] state_machine_generator: rename, and break up the init. of, "next_state"
Laszlo Ersek
lersek at redhat.com
Wed May 10 11:48:14 UTC 2023
We can shave off 17 characters by renaming "next_state" to "next", and
replacing its initialization with a standalone assignment.
Most impactful effect [lib/states.c]:
> int
> nbd_internal_enter_STATE_NEWSTYLE_OPT_STRUCTURED_REPLY_RECV_REPLY_PAYLOAD (
> struct nbd_handle *h, bool *blocked
> )
> {
> int r;
> - enum state next_state = STATE_NEWSTYLE_OPT_STRUCTURED_REPLY_RECV_REPLY_PAYLOAD;
> + enum state next;
>
> + next = STATE_NEWSTYLE_OPT_STRUCTURED_REPLY_RECV_REPLY_PAYLOAD;
> r = enter_STATE_NEWSTYLE_OPT_STRUCTURED_REPLY_RECV_REPLY_PAYLOAD (
> - h, &next_state, blocked
> + h, &next, blocked
> );
> - if (get_next_state (h) != next_state) {
> + if (get_next_state (h) != next) {
> debug (h, "transition: %s -> %s",
> "NEWSTYLE.OPT_STRUCTURED_REPLY.RECV_REPLY_PAYLOAD",
> - nbd_internal_state_short_string (next_state));
> - set_next_state (h, next_state);
> + nbd_internal_state_short_string (next));
> + set_next_state (h, next);
> }
> return r;
> }
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2172516
Signed-off-by: Laszlo Ersek <lersek at redhat.com>
---
generator/state_machine_generator.ml | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/generator/state_machine_generator.ml b/generator/state_machine_generator.ml
index af07e4ff3966..274e290952c1 100644
--- a/generator/state_machine_generator.ml
+++ b/generator/state_machine_generator.ml
@@ -373,16 +373,17 @@ let
pr ")\n";
pr "{\n";
pr " int r;\n";
- pr " enum state next_state = %s;\n" state_enum;
+ pr " enum state next;\n";
pr "\n";
+ pr " next = %s;\n" state_enum;
pr " r = enter_%s (\n" state_enum;
- pr " h, &next_state, blocked\n";
+ pr " h, &next, blocked\n";
pr " );\n";
- pr " if (get_next_state (h) != next_state) {\n";
+ pr " if (get_next_state (h) != next) {\n";
pr " debug (h, \"transition: %%s -> %%s\",\n";
pr " \"%s\",\n" display_name;
- pr " nbd_internal_state_short_string (next_state));\n";
- pr " set_next_state (h, next_state);\n";
+ pr " nbd_internal_state_short_string (next));\n";
+ pr " set_next_state (h, next);\n";
pr " }\n";
pr " return r;\n";
pr "}\n";
More information about the Libguestfs
mailing list