[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/11] Add operations to qlist to allow it to be
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH 02/11] Add operations to qlist to allow it to be used as a stack |
Date: |
Thu, 12 Nov 2009 15:20:50 -0200 |
On Thu, 12 Nov 2009 11:13:45 -0600
Anthony Liguori <address@hidden> wrote:
> Kevin Wolf wrote:
> > Unfortunately. There are places where such comments could be a good
> > specification on what an interface is actually meant to work like
> > (particularly in error cases). Currently you often can't tell if the
> > implementation or the caller of a function is buggy.
> >
> > Not sure if they are really useful for the simple qlist.c functions (but
> > even there the function name does not tell me what it's doing with NULL
> > parameters), but it might be helpful to have a general discussion about
> > it. I think in general qemu is poorly commented.
> >
>
> I agree, but I don't think the solution is forcing boiler plate
> commenting styles. I think what we could improve on is asking people to
> comment bits of code during review.
I've started adding comments like that because this is an API which
is probably going to be part of a library, as such it has to be properly
documented and very likely to be generated automatically by tools like
doxygen.
If we do this we should be consistent and document everything which
is public, even simple cases.
- [Qemu-devel] [PATCH 05/11] Add unit test for QFloat, (continued)
- [Qemu-devel] [PATCH 05/11] Add unit test for QFloat, Anthony Liguori, 2009/11/11
- [Qemu-devel] [PATCH 11/11] Add a unit test for JSON support, Anthony Liguori, 2009/11/11
- [Qemu-devel] [PATCH 06/11] Add a QBool type, Anthony Liguori, 2009/11/11
- [Qemu-devel] [PATCH 03/11] Allow strings to grow in size, Anthony Liguori, 2009/11/11
- [Qemu-devel] [PATCH 08/11] Add a JSON message boundary identifier, Anthony Liguori, 2009/11/11
- [Qemu-devel] [PATCH 02/11] Add operations to qlist to allow it to be used as a stack, Anthony Liguori, 2009/11/11
[Qemu-devel] [PATCH 04/11] Add a QFloat datatype, Anthony Liguori, 2009/11/11
[Qemu-devel] [PATCH 10/11] Add a QObject JSON wrapper, Anthony Liguori, 2009/11/11
[Qemu-devel] [PATCH 07/11] Add a lexer for JSON, Anthony Liguori, 2009/11/11