RPM Package Manager, CVS Repository
  http://rpm5.org/cvs/
  ____________________________________________________________________________

  Server: rpm5.org                         Name:   Ralf S. Engelschall
  Root:   /v/rpm/cvs                       Email:  [EMAIL PROTECTED]
  Module: popt                             Date:   14-Jun-2007 10:08:30
  Branch: HEAD                             Handle: 2007061409083000

  Modified files:
    popt                    popthelp.c

  Log:
    the POPT_ARG_CALLBACK argument 'displayArgs' is already a function as
    in ISO C the plain use of a function name always coerces into a
    function pointer. The use oft the '&' is both confusing and useless
    (and perhaps some compilers might even at least warn about this bad
    usage).

  Summary:
    Revision    Changes     Path
    1.56        +2  -2      popt/popthelp.c
  ____________________________________________________________________________

  patch -p0 <<'@@ .'
  Index: popt/popthelp.c
  ============================================================================
  $ cvs diff -u -r1.55 -r1.56 popthelp.c
  --- popt/popthelp.c   14 Jun 2007 08:00:39 -0000      1.55
  +++ popt/popthelp.c   14 Jun 2007 08:08:30 -0000      1.56
  @@ -64,7 +64,7 @@
   /[EMAIL PROTECTED]@*/
   /[EMAIL PROTECTED]@*/ /[EMAIL PROTECTED]@*/
   struct poptOption poptHelpOptions[] = {
  -  { NULL, '\0', POPT_ARG_CALLBACK, (void *)&displayArgs, 0, NULL, NULL },
  +  { NULL, '\0', POPT_ARG_CALLBACK, (void *)displayArgs, 0, NULL, NULL },
     { "help", '?', 0, NULL, (int)'?', N_("Show this help message"), NULL },
     { "usage", '\0', 0, NULL, (int)'u', N_("Display brief usage message"), 
NULL },
       POPT_TABLEEND
  @@ -75,7 +75,7 @@
   /[EMAIL PROTECTED]@*/
     { NULL, '\0', POPT_ARG_INTL_DOMAIN, PACKAGE, 0, NULL, NULL},
   /[EMAIL PROTECTED]@*/
  -  { NULL, '\0', POPT_ARG_CALLBACK, (void *)&displayArgs, 0, NULL, NULL },
  +  { NULL, '\0', POPT_ARG_CALLBACK, (void *)displayArgs, 0, NULL, NULL },
     { "help", '?', 0, NULL, (int)'?', N_("Show this help message"), NULL },
     { "usage", '\0', 0, NULL, (int)'u', N_("Display brief usage message"), 
NULL },
   #ifdef       NOTYET
  @@ .
______________________________________________________________________
RPM Package Manager                                    http://rpm5.org
CVS Sources Repository                                rpm-cvs@rpm5.org

Reply via email to