On Thu, Jun 2, 2011 at 6:13 AM, Andreas Färber <andreas.faer...@web.de> wrote: > Am 02.06.2011 um 04:21 schrieb Alexandre Raymond: > >> Fix the following warning by including signal.h directly in qemu-common.h >> ----8<---- >> iohandler.c: In function ‘qemu_init_child_watch’: >> iohandler.c:172: warning: implicit declaration of function ‘sigaction’ >> iohandler.c:172: warning: nested extern declaration of ‘sigaction’ >> ----8<---- >> >> Signed-off-by: Alexandre Raymond <cerb...@gmail.com> > > Tested-by: Andreas Färber <andreas.faer...@web.de> > > Why in qemu-common.h and not in iohandler.c though? > If we put it into qemu-common.h, you should remove other inclusions of > signal.h.
Well, I was simply following Anthony's advice from "[PATCH] #include cleanlines" : "The idea behind qemu-common.h is to avoid direct includes to help with portability." Alexandre