From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33007) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZLEEZ-0002QT-QT for qemu-devel@nongnu.org; Fri, 31 Jul 2015 13:36:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZLEER-0001NG-8r for qemu-devel@nongnu.org; Fri, 31 Jul 2015 13:36:39 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54430) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZLEER-0001N3-3K for qemu-devel@nongnu.org; Fri, 31 Jul 2015 13:36:31 -0400 Date: Fri, 31 Jul 2015 18:36:26 +0100 From: "Daniel P. Berrange" Message-ID: <20150731173626.GX2392@redhat.com> References: <1437583451-9677-1-git-send-email-peter.maydell@linaro.org> <1437583451-9677-11-git-send-email-peter.maydell@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1437583451-9677-11-git-send-email-peter.maydell@linaro.org> Subject: Re: [Qemu-devel] [PATCH 10/12] timer.h: Don't include qemu-common.h Reply-To: "Daniel P. Berrange" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: qemu-devel@nongnu.org, patches@linaro.org On Wed, Jul 22, 2015 at 05:44:09PM +0100, Peter Maydell wrote: > Include the specific headers we need, rather than qemu-common.h. > > Signed-off-by: Peter Maydell > --- > include/qemu/timer.h | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/include/qemu/timer.h b/include/qemu/timer.h > index 9939246..6205024 100644 > --- a/include/qemu/timer.h > +++ b/include/qemu/timer.h > @@ -1,8 +1,9 @@ > #ifndef QEMU_TIMER_H > #define QEMU_TIMER_H > > +#include "osdep.h" > +#include "glib-compat.h" I think there's probably a reasonable argument for glib-compat.h to be part of osdep.h as essentially all source files use glib now and so could potentially require the compat definitions. As mentioned in the cover letter, my preference would be for the osdep.h header to be mandated in all the .c files, which would mean we wouldn't need to add it here. Looks like the same points apply to your patches 11 & 12, so I won't repeat it again in those. > #include "qemu/typedefs.h" > -#include "qemu-common.h" > #include "qemu/notify.h" > #include "qemu/host-utils.h" Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|