[Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-14 Thread Luiz Capitulino
From: Luiz Capitulino This moves compiler related macros from qemu-common.h to compiler.h. The reason for this change is that there are simple header files that depend only on the compiler macros, so including qemu-common.h is overkill. Besides, qemu-common.h is bloated and will benefit from so

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-14 Thread malc
On Thu, 14 Jul 2011, Luiz Capitulino wrote: > On Thu, 14 Jul 2011 18:15:57 +0400 (MSD) > malc wrote: > > > On Thu, 14 Jul 2011, Luiz Capitulino wrote: > > > > > On Wed, 13 Jul 2011 22:39:07 +0200 > > > Stefan Weil wrote: > > > > > > > Am 13.07.2011 18:27, schrieb Luiz Capitulino: > > > > > On

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-14 Thread Luiz Capitulino
On Thu, 14 Jul 2011 18:15:57 +0400 (MSD) malc wrote: > On Thu, 14 Jul 2011, Luiz Capitulino wrote: > > > On Wed, 13 Jul 2011 22:39:07 +0200 > > Stefan Weil wrote: > > > > > Am 13.07.2011 18:27, schrieb Luiz Capitulino: > > > > On Wed, 13 Jul 2011 16:04:52 +0100 > > > > Peter Maydell wrote: >

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-14 Thread malc
On Thu, 14 Jul 2011, Luiz Capitulino wrote: > On Wed, 13 Jul 2011 22:39:07 +0200 > Stefan Weil wrote: > > > Am 13.07.2011 18:27, schrieb Luiz Capitulino: > > > On Wed, 13 Jul 2011 16:04:52 +0100 > > > Peter Maydell wrote: > > > > > >> On 13 July 2011 15:23, Luiz Capitulino wrote: > > >>> diff

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-14 Thread Luiz Capitulino
On Wed, 13 Jul 2011 22:39:07 +0200 Stefan Weil wrote: > Am 13.07.2011 18:27, schrieb Luiz Capitulino: > > On Wed, 13 Jul 2011 16:04:52 +0100 > > Peter Maydell wrote: > > > >> On 13 July 2011 15:23, Luiz Capitulino wrote: > >>> diff --git a/compiler.h b/compiler.h > >>> new file mode 100644 > >>

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-14 Thread Peter Maydell
On 13 July 2011 17:27, Luiz Capitulino wrote: > he asked me to remove the license text altogether (which makes this > public domain?) Saying nothing is definitely not putting something into the public domain, it's just leaving the recipient to guess (and for safety you generally have to assume th

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-13 Thread Stefan Weil
Am 13.07.2011 18:27, schrieb Luiz Capitulino: On Wed, 13 Jul 2011 16:04:52 +0100 Peter Maydell wrote: On 13 July 2011 15:23, Luiz Capitulino wrote: diff --git a/compiler.h b/compiler.h new file mode 100644 index 000..2dfc2c6 --- /dev/null +++ b/compiler.h @@ -0,0 +1,31 @@ +#ifndef COMPIL

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-13 Thread Luiz Capitulino
On Wed, 13 Jul 2011 16:04:52 +0100 Peter Maydell wrote: > On 13 July 2011 15:23, Luiz Capitulino wrote: > > diff --git a/compiler.h b/compiler.h > > new file mode 100644 > > index 000..2dfc2c6 > > --- /dev/null > > +++ b/compiler.h > > @@ -0,0 +1,31 @@ > > +#ifndef COMPILER_H > > +#define CO

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-13 Thread Luiz Capitulino
On Wed, 13 Jul 2011 18:37:31 +0400 (MSD) malc wrote: > On Wed, 13 Jul 2011, Luiz Capitulino wrote: > > > From: Luiz Capitulino > > > > This moves compiler related macros from qemu-common.h to compiler.h. > > > > The reason for this change is that there are simple header files that > > depend

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-13 Thread malc
On Wed, 13 Jul 2011, Luiz Capitulino wrote: > On Wed, 13 Jul 2011 18:37:31 +0400 (MSD) > malc wrote: > > > On Wed, 13 Jul 2011, Luiz Capitulino wrote: > > > > > From: Luiz Capitulino > > > > > > This moves compiler related macros from qemu-common.h to compiler.h. > > > > > > The reason for t

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-13 Thread Peter Maydell
On 13 July 2011 15:23, Luiz Capitulino wrote: > diff --git a/compiler.h b/compiler.h > new file mode 100644 > index 000..2dfc2c6 > --- /dev/null > +++ b/compiler.h > @@ -0,0 +1,31 @@ > +#ifndef COMPILER_H > +#define COMPILER_H If we don't have a rule that says that all new source files should

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-13 Thread malc
On Wed, 13 Jul 2011, Luiz Capitulino wrote: > From: Luiz Capitulino > > This moves compiler related macros from qemu-common.h to compiler.h. > > The reason for this change is that there are simple header files that > depend only on the compiler macros, so including qemu-common.h is overkill. >

[Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-13 Thread Luiz Capitulino
From: Luiz Capitulino This moves compiler related macros from qemu-common.h to compiler.h. The reason for this change is that there are simple header files that depend only on the compiler macros, so including qemu-common.h is overkill. Besides, qemu-common.h is bloated and will benefit from so

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-12 Thread Luiz Capitulino
On Wed, 13 Jul 2011 00:43:42 +0400 (MSD) malc wrote: > On Tue, 12 Jul 2011, Luiz Capitulino wrote: > > > On Wed, 13 Jul 2011 00:31:56 +0400 (MSD) > > malc wrote: > > > > > On Tue, 12 Jul 2011, Luiz Capitulino wrote: > > > > > > > From: Luiz Capitulino > > > > > > > > This moves compiler rel

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-12 Thread malc
On Tue, 12 Jul 2011, Luiz Capitulino wrote: > On Wed, 13 Jul 2011 00:31:56 +0400 (MSD) > malc wrote: > > > On Tue, 12 Jul 2011, Luiz Capitulino wrote: > > > > > From: Luiz Capitulino > > > > > > This moves compiler related macros from qemu-common.h to compiler.h. > > > > > > The reason for t

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-12 Thread Luiz Capitulino
On Wed, 13 Jul 2011 00:31:56 +0400 (MSD) malc wrote: > On Tue, 12 Jul 2011, Luiz Capitulino wrote: > > > From: Luiz Capitulino > > > > This moves compiler related macros from qemu-common.h to compiler.h. > > > > The reason for this change is that there are simple header files that > > depend

Re: [Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-12 Thread malc
On Tue, 12 Jul 2011, Luiz Capitulino wrote: > From: Luiz Capitulino > > This moves compiler related macros from qemu-common.h to compiler.h. > > The reason for this change is that there are simple header files that > depend only on the compiler macros, so including qemu-common.h is overkill. >

[Qemu-devel] [PATCH 1/2] Introduce compiler.h header file

2011-07-12 Thread Luiz Capitulino
From: Luiz Capitulino This moves compiler related macros from qemu-common.h to compiler.h. The reason for this change is that there are simple header files that depend only on the compiler macros, so including qemu-common.h is overkill. Besides, qemu-common.h is bloated and will benefit from so