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.