[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [bug-inetutils] ping/ping6 removing common code for review - 5
From: |
Alfred M\. Szmidt |
Subject: |
Re: [bug-inetutils] ping/ping6 removing common code for review - 5 |
Date: |
Tue, 14 Oct 2008 21:09:19 +0200 |
I have committed 4 old already reviewed patch with fixed ChangeLog
messages and all suggested changes:
Thank you.
* ping/ping_impl.h, ping/ping.h, ping/ping_common.h,
ping/ping6.h (ping_stat, PEV_RESPONSE, PEV_DUPLICATE, PEV_NOECHO,
PING_INTERVAL, PING_CKTABSIZE, MAXWAIT, OPT_FLOOD, OPT_INTERVAL,
OPT_NUMERIC, OPT_QUIET, OPT_RROUTE, OPT_VERBOSE, PING_TIMING,
PING_HEADER_LEN, PING_DATALEN): Moved all these common macros and
structures from ping_impl.h , ping6.h and ping_ping.h to
ping_common.h. Removed PING_INTERVAL.
* ping/ping6.c (ping_run): Used PING_SET_INTERVAL similar to
libping.c and used PING_DEFAULT_INTERVAL in place of PING_INTERVAL.
* ping/libping.c (ping_set_datalen): Removed.
* ping/libping.c, ping/ping_common.c (_ping_freebuf, ping_unset_data):
Moved _ping_freebuf, ping_unset_data from libping.c to ping_common.c
* ping/ping6.c (ping_run): Free memory with call to ping_unset_data.
Ok to commit (if you could, fix the small typographical things, two
spaces after a period, sentences end with a period, capital letter at
the beginning of the sentence, no space after comma, ...)
Thank you, I am looking forward to more patches that will clean up the
mess that is ping!