[pisa-src] Re: r1661 - trunk/pisabeacon/beaconmain.c

  • From: Diego Biurrun <diego@xxxxxxxxxx>
  • To: pisa-src@xxxxxxxxxxxxx
  • Date: Thu, 12 Nov 2009 14:15:43 +0100

On Thu, Nov 12, 2009 at 02:12:24PM +0100, Thomas Jansen wrote:
> 
> Log:
> Fixed commit 1660. quitcode became unused causing a warning.
> 
> --- trunk/pisabeacon/beaconmain.c     Thu Nov 12 14:05:16 2009        (r1660)
> +++ trunk/pisabeacon/beaconmain.c     Thu Nov 12 14:12:24 2009        (r1661)
> @@ -87,7 +89,7 @@
>   *
>   * @param quitcode   signal quit code
>   */
> -static void plb_quit(int quitcode) {
> +static void plb_quit(UNUSED int quitcode) {
>       printf("Quitting PISA location server daemon...\n");
>       exit(EXIT_SUCCESS);
>  }

Why not drop the function argument if it is unused?  Seems more sensible
to me...

Diego

Other related posts: