[Top][All Lists]

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

Re: [TRUNCATED MESSAGE] [bug-inetutils] [PATCH] Fix compiler warnings

From: Alfred M. Szmidt
Subject: Re: [TRUNCATED MESSAGE] [bug-inetutils] [PATCH] Fix compiler warnings
Date: Thu, 03 Dec 2009 11:00:58 -0500

   This patch fixes any warning reported by gcc 4.3.4 (I haven't yet tried
   with newer versions).  All of them are trivial changes but given the
   patch size, I think it is better to postpone it after the release.

   Any comment?

I say push it.  We will have to fix any bugs that might arise anyway.

   diff --git a/ChangeLog b/ChangeLog
   index cd986cf..c3f004a 100644
   --- a/ChangeLog
   +++ b/ChangeLog
   @@ -1,3 +1,127 @@
   +2009-12-03  Giuseppe Scrivano  <address@hidden>
   +    * ftp/cmds.c (domap): Add braces around the else branch.
   +    (strup): Remove.

Please mention if strup is a function, or variable.

   diff --git a/ftpd/ftpcmd.y b/ftpd/ftpcmd.y
   index e52ff61..84be120 100644
   --- a/ftpd/ftpcmd.y
   +++ b/ftpd/ftpcmd.y
   @@ -995,7 +995,7 @@ yylex()
                                           /* NOTREACHED */
                                   state = p->state;
   -                            yylval.s = p->name;
   +                            yylval.s = (char*)p->name;

Space between cast and variable, same thing in some other places.

   diff --git a/libinetutils/utmp_logout.c b/libinetutils/utmp_logout.c
   index e055b0b..54947c1 100644
   --- a/libinetutils/utmp_logout.c
   +++ b/libinetutils/utmp_logout.c
   @@ -89,7 +89,7 @@ utmp_logout (char *line)

      strncpy (utx.ut_line, line, sizeof (utx.ut_line));

   -  if (ut = getutline (&utx))
   +  if ((ut = getutline (&utx)))

Better to move the assignment outside the if statement, i.e.

ut = getutline (&utx);
if (ut)

A few other places use the assignment in if statment variant, those
should be fixed as well.  You don't need to fix all the existing
cases, but atleast the ones you touched in this patch where you add
extra parens...  Unless you want to :-)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]