| Message ID | 1357579795-11770-1-git-send-email-peter.maydell@linaro.org |
|---|---|
| State | Accepted |
| Commit | 1d57db193f2eb619ccc9a60e76120379b757d9f2 |
| Headers | show |
On 7 January 2013 17:38, Stefan Weil <sw@weilnetz.de> wrote: > Am 07.01.2013 18:29, schrieb Peter Maydell: > >> Explicitly include string.h to avoid warnings under MacOS X/clang >> about implicit declarations of strerror() and strlen(). > Better: include qemu-common.h first and remove all > three standard includes. This would be reversing direction from commit 4d4922c33... I took the commit message in that commit to imply that we shouldn't be including qemu-common.h for standalone executables like qga. -- PMM
On Mon, Jan 07, 2013 at 05:50:14PM +0000, Peter Maydell wrote: > On 7 January 2013 17:38, Stefan Weil <sw@weilnetz.de> wrote: > > Am 07.01.2013 18:29, schrieb Peter Maydell: > > > >> Explicitly include string.h to avoid warnings under MacOS X/clang > >> about implicit declarations of strerror() and strlen(). > > > Better: include qemu-common.h first and remove all > > three standard includes. > > This would be reversing direction from commit 4d4922c33... > I took the commit message in that commit to imply that we > shouldn't be including qemu-common.h for standalone > executables like qga. Including qemu-common.h from .c files is OK, although I wouldn't do that in this specific case (I would include string.h instead). The problem with qemu-common.h is when it is included by header files, because qemu-common.h includes lots of other headers, easily leading to circular header dependencies.
On Mon, Jan 07, 2013 at 05:29:55PM +0000, Peter Maydell wrote: > Explicitly include string.h to avoid warnings under MacOS X/clang > about implicit declarations of strerror() and strlen(). > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> > --- > I assume under Linux these are implicitly dragged in via one of the > other headers. > > qga/channel-posix.c | 1 + > 1 file changed, 1 insertion(+) Thanks, applied to the trivial patches tree: https://github.com/stefanha/qemu/commits/trivial-patches Stefan
diff --git a/qga/channel-posix.c b/qga/channel-posix.c index d4fd628..ca9e4aa 100644 --- a/qga/channel-posix.c +++ b/qga/channel-posix.c @@ -4,6 +4,7 @@ #include <unistd.h> #include <fcntl.h> #include <stdlib.h> +#include <string.h> #include "qemu/osdep.h" #include "qemu/sockets.h" #include "qga/channel.h"
Explicitly include string.h to avoid warnings under MacOS X/clang about implicit declarations of strerror() and strlen(). Signed-off-by: Peter Maydell <peter.maydell@linaro.org> --- I assume under Linux these are implicitly dragged in via one of the other headers. qga/channel-posix.c | 1 + 1 file changed, 1 insertion(+)