Skip to contentSkip to conversation

libxl: add missing includes

PATCH In review Christoph Egger 2 participants 4 messages Feb 9, 2012 – Feb 21, 2012

lore
Share & tools
lore

Christoph Egger

author
#0 lore raw

PATCH 1/1 libxl: add missing includes

include <poll.h> for struct pollfd
include <time.h> for struct timeval
Fixes gcc complaints about implicit declaration.

Signed-off-by Christoph Egger <[email protected]>
Signature · 6 lines
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85689 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632
tools/libxl/libxl_event.h +2 -0
diff -r f27d5c83b05e tools/libxl/libxl_event.h
@@ -17,6 +17,8 @@
 #define LIBXL_EVENT_H
 
 #include <libxl.h>
+#include <poll.h>
+#include <time.h>
 
 /*======================================================================*/

_______________________________________________
Xen-devel mailing list
[email protected]
http://lists.xensource.com/xen-devel

#1 lore raw

Christoph Egger writes ("[Xen-devel] [PATCH] libxl: add missing includes"):

Quoted 4 lines

include <poll.h> for struct pollfd
include <time.h> for struct timeval
Fixes gcc complaints about implicit declaration.

According to SuS, it should be <sys/time.h>.

Ian.

author
#2 lore raw
Quoted 7 lines · Ian Jackson

Christoph Egger writes ("[Xen-devel] [PATCH] libxl: add missing includes"):

Quoted 4 lines

include<poll.h> for struct pollfd
include<time.h> for struct timeval
Fixes gcc complaints about implicit declaration.

According to SuS, it should be<sys/time.h>.

Yes, sorry. Should I resend?

Christoph

Signature · 6 lines
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85689 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632
#3 lore raw

Christoph Egger writes ("Re: [Xen-devel] [PATCH] libxl: add missing includes"):

Quoted 10 lines · Ian Jackson
Quoted 7 lines

Christoph Egger writes ("[Xen-devel] [PATCH] libxl: add missing includes"):

Quoted 4 lines

include<poll.h> for struct pollfd
include<time.h> for struct timeval
Fixes gcc complaints about implicit declaration.

According to SuS, it should be<sys/time.h>.

Yes, sorry. Should I resend?

I have fixed it up and committed it.

Thanks,
Ian.