From 7868f017830fa28fb1cfeca29d48da60475bf6a8 Mon Sep 17 00:00:00 2001 From: Martin Kletzander Date: Wed, 22 Jul 2015 11:01:58 +0200 Subject: [PATCH] admin: Tiny cleanups First hunk changes the use of srcdir to top_srcdir so it complies with other rules in the Makefile. Second one removes the need of remote_protocol.h in admin_protocol.h as it was suggested and worked in, but this one line was missed apparently. Last one just removes the 'remote' naming from admin protocol specification, just so it's cleaner. Signed-off-by: Martin Kletzander --- daemon/Makefile.am | 4 ++-- src/admin/admin_protocol.x | 6 ++---- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/daemon/Makefile.am b/daemon/Makefile.am index d3fc029f05..59bc4d4edd 100644 --- a/daemon/Makefile.am +++ b/daemon/Makefile.am @@ -101,9 +101,9 @@ qemu_dispatch.h: $(top_srcdir)/src/rpc/gendispatch.pl \ --mode=server qemu QEMU $(QEMU_PROTOCOL) \ > $(srcdir)/qemu_dispatch.h -admin_dispatch.h: $(srcdir)/../src/rpc/gendispatch.pl \ +admin_dispatch.h: $(top_srcdir)/src/rpc/gendispatch.pl \ $(ADMIN_PROTOCOL) - $(AM_V_GEN)$(PERL) -w $(srcdir)/../src/rpc/gendispatch.pl \ + $(AM_V_GEN)$(PERL) -w $(top_srcdir)/src/rpc/gendispatch.pl \ --mode=server admin ADMIN $(ADMIN_PROTOCOL) \ > $(srcdir)/admin_dispatch.h diff --git a/src/admin/admin_protocol.x b/src/admin/admin_protocol.x index 1a2e94ee86..cfc92ff748 100644 --- a/src/admin/admin_protocol.x +++ b/src/admin/admin_protocol.x @@ -22,18 +22,16 @@ * Author: Martin Kletzander */ -%#include "remote_protocol.h" - /*----- Data types. -----*/ /* Length of long, but not unbounded, strings. * This is an arbitrary limit designed to stop the decoder from trying * to allocate unbounded amounts of memory when fed with a bad message. */ -const REMOTE_STRING_MAX = 4194304; +const ADMIN_STRING_MAX = 4194304; /* A long string, which may NOT be NULL. */ -typedef string admin_nonnull_string; +typedef string admin_nonnull_string; /* A long string, which may be NULL. */ typedef admin_nonnull_string *admin_string; -- GitLab