Re: [PATCH v3 4/8] lib: document existing free functions

2022-06-22 Thread Stephen Hemminger
On Wed, 22 Jun 2022 11:23:04 +0200 Thomas Monjalon wrote: > 28/02/2022 18:08, Stephen Hemminger: > > On Mon, 28 Feb 2022 09:42:47 + > > Bruce Richardson wrote: > > > > > On Sun, Feb 27, 2022 at 09:48:57PM +0100, Thomas Monjalon wrote: > > > > 20/02/2022 19:21, Stephen Hemminger: > >

Re: [PATCH v3 4/8] lib: document existing free functions

2022-06-22 Thread Thomas Monjalon
28/02/2022 18:08, Stephen Hemminger: > On Mon, 28 Feb 2022 09:42:47 + > Bruce Richardson wrote: > > > On Sun, Feb 27, 2022 at 09:48:57PM +0100, Thomas Monjalon wrote: > > > 20/02/2022 19:21, Stephen Hemminger: > > > > + * If NULL then, the function does nothing. > > > > > > I'm not Eng

Re: [PATCH v3 4/8] lib: document existing free functions

2022-02-28 Thread Stephen Hemminger
On Mon, 28 Feb 2022 09:42:47 + Bruce Richardson wrote: > On Sun, Feb 27, 2022 at 09:48:57PM +0100, Thomas Monjalon wrote: > > 20/02/2022 19:21, Stephen Hemminger: > > > + * If NULL then, the function does nothing. > > > > I'm not English native, but I thought it should be one of these

Re: [PATCH v3 4/8] lib: document existing free functions

2022-02-28 Thread Bruce Richardson
On Sun, Feb 27, 2022 at 09:48:57PM +0100, Thomas Monjalon wrote: > 20/02/2022 19:21, Stephen Hemminger: > > + * If NULL then, the function does nothing. > > I'm not English native, but I thought it should be one of these 2 forms: > - If NULL, the function does nothing. > - If NULL th

Re: [PATCH v3 4/8] lib: document existing free functions

2022-02-27 Thread Thomas Monjalon
20/02/2022 19:21, Stephen Hemminger: > + * If NULL then, the function does nothing. I'm not English native, but I thought it should be one of these 2 forms: - If NULL, the function does nothing. - If NULL then the function does nothing.

[PATCH v3 4/8] lib: document existing free functions

2022-02-20 Thread Stephen Hemminger
These functions all accept NULL as parameter. Signed-off-by: Stephen Hemminger --- lib/bitratestats/rte_bitrate.h | 1 + lib/compressdev/rte_comp.h | 1 + lib/cryptodev/rte_crypto.h | 1 + lib/eal/include/rte_interrupts.h | 4 +++- lib/efd/rte_efd.h| 1 + lib/eventd