Laszlo Ersek
2023-Apr-13 14:03 UTC
[Libguestfs] [nbdkit PATCH 17/17] server: wrap source code at 80 chars
Wrap the source code at 80 characters. Extract typedefs "plugin_init_function" and "filter_init_function". Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2172516 Signed-off-by: Laszlo Ersek <lersek at redhat.com> --- server/internal.h | 7 +++++-- server/filters.c | 2 +- server/main.c | 6 ++++-- server/plugins.c | 2 +- 4 files changed, 11 insertions(+), 6 deletions(-) diff --git a/server/internal.h b/server/internal.h index 4b6e29154e10..2aff4cfff91d 100644 --- a/server/internal.h +++ b/server/internal.h @@ -508,14 +508,17 @@ extern int backend_cache (struct context *c, __attribute__ ((__nonnull__ (1, 5))); /* plugins.c */ +typedef struct nbdkit_plugin *(*plugin_init_function) (void); extern struct backend *plugin_register (size_t index, const char *filename, - void *dl, struct nbdkit_plugin *(*plugin_init) (void)) + void *dl, + plugin_init_function plugin_init) __attribute__ ((__nonnull__ (2, 3, 4))); /* filters.c */ +typedef struct nbdkit_filter *(*filter_init_function) (void); extern struct backend *filter_register (struct backend *next, size_t index, const char *filename, void *dl, - struct nbdkit_filter *(*filter_init) (void)) + filter_init_function filter_init) __attribute__ ((__nonnull__ (1, 3, 4, 5))); /* locks.c */ diff --git a/server/filters.c b/server/filters.c index f32a9b46f6f2..b667754b217d 100644 --- a/server/filters.c +++ b/server/filters.c @@ -660,7 +660,7 @@ static struct backend filter_functions = { /* Register and load a filter. */ struct backend * filter_register (struct backend *next, size_t index, const char *filename, - void *dl, struct nbdkit_filter *(*filter_init) (void)) + void *dl, filter_init_function filter_init) { struct backend_filter *f; const struct nbdkit_filter *filter; diff --git a/server/main.c b/server/main.c index 631401b72712..c3b9bf3848ac 100644 --- a/server/main.c +++ b/server/main.c @@ -79,8 +79,10 @@ #endif static char *make_random_fifo (void); -static struct backend *open_plugin_so (size_t i, const char *filename, int short_name); -static struct backend *open_filter_so (struct backend *next, size_t i, const char *filename, int short_name); +static struct backend *open_plugin_so (size_t i, const char *filename, + int short_name); +static struct backend *open_filter_so (struct backend *next, size_t i, + const char *filename, int short_name); static void start_serving (void); static void write_pidfile (void); static bool is_config_key (const char *key, size_t len); diff --git a/server/plugins.c b/server/plugins.c index 1faac348045e..85eac76c45e7 100644 --- a/server/plugins.c +++ b/server/plugins.c @@ -908,7 +908,7 @@ static struct backend plugin_functions = { /* Register and load a plugin. */ struct backend * plugin_register (size_t index, const char *filename, - void *dl, struct nbdkit_plugin *(*plugin_init) (void)) + void *dl, plugin_init_function plugin_init) { struct backend_plugin *p; const struct nbdkit_plugin *plugin;
Richard W.M. Jones
2023-Apr-13 14:23 UTC
[Libguestfs] [nbdkit PATCH 17/17] server: wrap source code at 80 chars
This series looks fine to me. 'make check' still passes? It contains some --help validation tests. I was preparing to do a 1.34 (new stable) release, so it's best to get this in sooner rather than later, otherwise it will miss the release. I will look at adding a test for nbdkit-floppy-plugin + size. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com nbdkit - Flexible, fast NBD server with plugins https://gitlab.com/nbdkit/nbdkit