On Wed, Jan 23, 2019 at 04:25:33AM +0530, Isira Seneviratne wrote:
>

> From 5e4bd5880b74c8340a0b41fd0867c8ee70bd2817 Mon Sep 17 00:00:00 2001
> From: Isira-Seneviratne <isirasen96@gmail.com>
> Date: Sat, 5 Jan 2019 10:46:15 +0530
> Subject: [PATCH] ping: Add Ctrl+C event handling
>
> Signed-off-by: Isira-Seneviratne <isirasen96@gmail.com>
> ---
>  programs/ping/ping_main.c | 48 +++++++++++++++++++++++++++++++++++----
>  1 file changed, 44 insertions(+), 4 deletions(-)
>
> diff --git a/programs/ping/ping_main.c b/programs/ping/ping_main.c
> index 9c4f23959e..5e7e7542bc 100644
> --- a/programs/ping/ping_main.c
> +++ b/programs/ping/ping_main.c
> @@ -39,8 +39,35 @@

> +/* Displays ping statistics when the interrupt signal is sent to the program */
> +BOOL WINAPI ping_interrupted(DWORD dwCtrlType)
> +{
> +    switch (dwCtrlType)
> +    {
> +        case CTRL_C_EVENT:
> +            printf("\nPing statistics for %s\n", ip);
> +            printf("\tPackets: Sent = %d, Received = %d, Lost = %d (%.0f%% loss)\n",
> +                i, rec, lost, (float) lost / i * 100);
> +            if (rec != 0)
> +            {
> +                avg /= rec;
> +                printf("Approximate round trip times in milli-seconds:\n");
> +                printf("\tMinimum = %dms, Maximum = %dms, Average = %.0fms\n",
> +                    min, max, avg);
> +            }
> +            exit(0);
> +        default:
> +            return FALSE;
> +    }
> +}

So the printing code now appears twice in the file, here and at the
bottom of main().  What you should do is to move the printing code to
a separate function that can be called from both places.

Huw.