[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH] CODING_STYLE: Reserve qemu_ prefix for libr
From: |
Jamie Lokier |
Subject: |
Re: [Qemu-devel] Re: [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers |
Date: |
Thu, 11 Mar 2010 23:08:34 +0000 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
Anthony Liguori wrote:
> On 03/11/2010 08:48 AM, Avi Kivity wrote:
> >Signed-off-by: Avi Kivity<address@hidden>
> >---
> > CODING_STYLE | 3 +++
> > 1 files changed, 3 insertions(+), 0 deletions(-)
> >
> >diff --git a/CODING_STYLE b/CODING_STYLE
> >index a579cb1..92036f3 100644
> >--- a/CODING_STYLE
> >+++ b/CODING_STYLE
> >@@ -49,6 +49,9 @@ and is therefore likely to be changed.
> > Typedefs are used to eliminate the redundant 'struct' keyword. It is the
> > QEMU coding style.
> >
> >+When wrapping standard library functions, use the prefix qemu_ to alert
> >+readers that they are seeing a wrapped version; otherwise avoid this
> >prefix.
> >+
> >
>
> Acked-by: Anthony Liguori <address@hidden>
>
> But I'd like to also see others agree with this.
I agree - not that I've contributed any code, mind you, but I do read
bits of it from time to time.
However if qemu is ever made into a library used by other
applications, i.e. libqemu.a or libqemu.so, then you might decide
application-visible functions should have the prefix
-- Jamie
- Re: [Qemu-devel] [PATCH 1/7] Add support for generic notifier lists, (continued)
Re: [Qemu-devel] [PATCH 1/7] Add support for generic notifier lists, Avi Kivity, 2010/03/11
- Re: [Qemu-devel] [PATCH 1/7] Add support for generic notifier lists, Anthony Liguori, 2010/03/11
- Re: [Qemu-devel] [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers, Markus Armbruster, 2010/03/11
- [Qemu-devel] Re: [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers, Juan Quintela, 2010/03/11
- Re: [Qemu-devel] [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers, Aurelien Jarno, 2010/03/12
- Re: [Qemu-devel] [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers, Paul Brook, 2010/03/12
- Re: [Qemu-devel] [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers, Edgar E. Iglesias, 2010/03/12
- Re: [Qemu-devel] [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers, Avi Kivity, 2010/03/13
- Re: [Qemu-devel] [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers, Edgar E. Iglesias, 2010/03/13
Re: [Qemu-devel] [PATCH] CODING_STYLE: Reserve qemu_ prefix for library wrappers, Anthony Liguori, 2010/03/22