qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] 9pfs: drop Doxygen format from qemu_dirent_dup() API comment


From: Greg Kurz
Subject: Re: [PATCH] 9pfs: drop Doxygen format from qemu_dirent_dup() API comment
Date: Thu, 3 Mar 2022 12:29:53 +0100

On Thu, 03 Mar 2022 11:55:36 +0100
Christian Schoenebeck <qemu_oss@crudebyte.com> wrote:

> On Donnerstag, 3. März 2022 10:53:32 CET Greg Kurz wrote:
> > On Wed, 2 Mar 2022 19:30:39 +0100
> > 
> > Christian Schoenebeck <qemu_oss@crudebyte.com> wrote:
> > > API doc comments in QEMU are supposed to be in kerneldoc format, so drop
> > > occurrences of "@c" which is Doxygen format for fixed-width text.
> > > 
> > > Link:
> > > https://lore.kernel.org/qemu-devel/CAFEAcA89+ENOM6x19OEF53Kd2DWkhN5SN21Va
> > > 0D7yepJSa3Jyg@mail.gmail.com/ Based-on:
> > > <E1nP9Oz-00043L-KJ@lizzy.crudebyte.com>
> > > Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
> > > ---
> > 
> > Sorry for not catching these during previous reviews :-\
> > 
> > Reviewed-by: Greg Kurz <groug@kaod.org>
> > 
> > FWIW it seems some more de-doxygenation is needed :
> > 
> > fsdev/p9array.h: * Consider the following user struct @c Foo which shall be
> > used as scalar fsdev/p9array.h: * @code
> > fsdev/p9array.h: * and assume it has the following function to free memory
> > allocated by @c Foo fsdev/p9array.h: * @code
> > fsdev/p9array.h: * @code
> > fsdev/p9array.h: * @code
> > fsdev/p9array.h: * @code
> > hw/9pfs/9p.h:     * Lenght of the affix, that is how many (of the lowest)
> > bits of @c value extra nit  ^^
> > hw/9pfs/codir.c: * @note You must @b ALWAYS call @c
> > v9fs_free_dirents(entries) after calling
> 
> OK, I send more patches to wipe these as well today.
> 
> Greg, if something is tagged as junk email at your end, please remove it from 
> the subject line, otherwise I will not receive your reply. It was pure luck 
> that I noticed your message on the archives this time.
> 

Sure... I usually do but I forgot this time. Sorry for that.

Since you seem to be the only affected person which I receive
mail from, I'll try to automate this on my side.

> Best regards,
> Christian Schoenebeck
> 
> 




reply via email to

[Prev in Thread] Current Thread [Next in Thread]