Messages in this thread | | | Subject | Re: [patch 2/3] flex_array: fix flex_array_free_parts comment | From | Dave Hansen <> | Date | Mon, 17 Aug 2009 17:06:46 -0700 |
| |
On Mon, 2009-08-17 at 16:46 -0700, David Rientjes wrote: > flex_array_free_parts() does not take `src' or `element_nr' formals, so > remove their respective comments. > > Cc: Dave Hansen <dave@linux.vnet.ibm.com> > Signed-off-by: David Rientjes <rientjes@google.com> > --- > lib/flex_array.c | 3 --- > 1 files changed, 0 insertions(+), 3 deletions(-) > > diff --git a/lib/flex_array.c b/lib/flex_array.c > --- a/lib/flex_array.c > +++ b/lib/flex_array.c > @@ -122,9 +122,6 @@ static int fa_element_to_part_nr(struct flex_array *fa, int element_nr) > > /** > * flex_array_free_parts - just free the second-level pages > - * @src: address of data to copy into the array > - * @element_nr: index of the position in which to insert > - * the new element. > * > * This is to be used in cases where the base 'struct flex_array' > * has been statically allocated and should not be free.
Looks great to me.
Signed-off-by: Dave Hansen <dave@linux.vnet.ibm.com>
-- Dave
| |