diff options
author | Richard W.M. Jones <rjones@redhat.com> | 2012-09-17 13:31:45 +0100 |
---|---|---|
committer | Richard W.M. Jones <rjones@redhat.com> | 2012-09-17 13:32:49 +0100 |
commit | c387b69cba8f145438188b2b4567abbc9580ade0 (patch) | |
tree | c847ebcb65d32f896783d3bb4d219817b932fc47 | |
parent | 176bd182d3f94fe7e774b8af0b56c014f639d6c7 (diff) | |
download | libguestfs-c387b69cba8f145438188b2b4567abbc9580ade0.tar.gz libguestfs-c387b69cba8f145438188b2b4567abbc9580ade0.tar.xz libguestfs-c387b69cba8f145438188b2b4567abbc9580ade0.zip |
daemon: stubs: Move all arg decls to top of function.
For some reason, argument declarations were handled
inconsistently for strings and non-string arguments.
Non-string arguments were all declared at the top of the
function stub, where as string arguments were declared
in place.
Be consistent.
This is just code motion.
-rw-r--r-- | generator/daemon.ml | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/generator/daemon.ml b/generator/daemon.ml index 06ee8120..a807be2a 100644 --- a/generator/daemon.ml +++ b/generator/daemon.ml @@ -107,7 +107,7 @@ and generate_daemon_actions () = | Device n | Dev_or_Path n | Pathname n | String n - | Key n -> () + | Key n | OptString n -> pr " char *%s;\n" n | StringList n | DeviceList n -> pr " char **%s;\n" n | Bool n -> pr " int %s;\n" n @@ -178,7 +178,7 @@ and generate_daemon_actions () = pr " goto done;\n"; pr " }\n"; let pr_args n = - pr " char *%s = args.%s;\n" n n + pr " %s = args.%s;\n" n n in let pr_list_handling_code n = pr " %s = realloc (args.%s.%s_val,\n" n n n; |