Re: [PATCH] Fix compiler warnings in echo extension

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Bernhard Walle wrote:
Fixes

echo.c:24: warning: missing braces around initializer
echo.c:24: warning: (near initialization for 'command_table[0]')
echo.c:30: warning: return type defaults to 'int'
echo.c:38: warning: return type defaults to 'int'
echo.c: In function '_fini':
echo.c:38: warning: control reaches end of non-void function
echo.c: In function '_init':
echo.c:32: warning: control reaches end of non-void function


Signed-off-by: Bernhard Walle <bwalle@xxxxxxx>


Adding "-Wall -Wstrict-prototypes -Wmissing-prototypes" to the
echo.c build generated a couple more complaints beyond the
ones you found.  I also fixed a couple complaints in dminfo.c.

Note that sial.c generates quite a few as well, but I'm not
touching that one -- Luc, if you have any interest in doing
so, be my guest...

Queued for the next release.

Thanks,
  Dave

---
 extensions/echo.c |    9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

--- a/extensions/echo.c
+++ b/extensions/echo.c
@@ -21,21 +21,22 @@ void cmd_echo();     /* Declare the comm
 char *help_echo[];
static struct command_table_entry command_table[] = {
-        "echo", cmd_echo, help_echo, 0,           /* One or more commands, */
-        NULL,                                     /* terminated by NULL, */
+	{ "echo", cmd_echo, help_echo, 0 },    /* One or more commands, */
+	{ NULL }                               /* terminated by NULL, */
 };
-_init() /* Register the command set. */
+int _init() /* Register the command set. */
{ register_extension(command_table);
+	return 1;
 }
/* * The _fini() function is called if the shared object is unloaded. * If desired, perform any cleanups here. */
-_fini() { }
+void _fini() { }
/*
--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility


--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility

[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux