Re: [Qemu-trivial] [PATCH v2] libcacard: stop including qemu-common.h

[Paolo Bonzini <pbonzini@redhat.com>]

On 27/04/2015 15:27, Michael Tokarev wrote:
> From: Paolo Bonzini <pbonzini@redhat.com>
> 
> This is a small step towards making libcacard standalone.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
> ---
> v2: update vscclient.c too, use unistd.h on !WIN32 and getopt.h on *nix

Thanks very much Michael.  The vscclient.c parts look good, you may want
to add your changes before your S-o-b line.

Paolo

> 
>  libcacard/cac.c            | 5 ++++-
>  libcacard/card_7816.c      | 4 +++-
>  libcacard/event.c          | 2 +-
>  libcacard/vcard.c          | 4 +++-
>  libcacard/vcard_emul_nss.c | 2 +-
>  libcacard/vreader.c        | 4 +++-
>  libcacard/vscclient.c      | 8 +++++++-
>  7 files changed, 22 insertions(+), 7 deletions(-)
> 
> diff --git a/libcacard/cac.c b/libcacard/cac.c
> index f38fdce..bc84534 100644
> --- a/libcacard/cac.c
> +++ b/libcacard/cac.c
> @@ -5,7 +5,10 @@
>   * See the COPYING.LIB file in the top-level directory.
>   */
>  
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
> +#include <stdbool.h>
>  
>  #include "cac.h"
>  #include "vcard.h"
> diff --git a/libcacard/card_7816.c b/libcacard/card_7816.c
> index 814fa16..22fd334 100644
> --- a/libcacard/card_7816.c
> +++ b/libcacard/card_7816.c
> @@ -5,7 +5,9 @@
>   * See the COPYING.LIB file in the top-level directory.
>   */
>  
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>  
>  #include "vcard.h"
>  #include "vcard_emul.h"
> diff --git a/libcacard/event.c b/libcacard/event.c
> index 4c551e4..63f4057 100644
> --- a/libcacard/event.c
> +++ b/libcacard/event.c
> @@ -5,7 +5,7 @@
>   * See the COPYING.LIB file in the top-level directory.
>   */
>  
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>  
>  #include "vcard.h"
>  #include "vreader.h"
> diff --git a/libcacard/vcard.c b/libcacard/vcard.c
> index d140a8e..1a87208 100644
> --- a/libcacard/vcard.c
> +++ b/libcacard/vcard.c
> @@ -5,7 +5,9 @@
>   * See the COPYING.LIB file in the top-level directory.
>   */
>  
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>  
>  #include "vcard.h"
>  #include "vcard_emul.h"
> diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c
> index 950edee..6955f69 100644
> --- a/libcacard/vcard_emul_nss.c
> +++ b/libcacard/vcard_emul_nss.c
> @@ -25,7 +25,7 @@
>  #include <prthread.h>
>  #include <secerr.h>
>  
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>  
>  #include "vcard.h"
>  #include "card_7816t.h"
> diff --git a/libcacard/vreader.c b/libcacard/vreader.c
> index 0315dd8..9725f46 100644
> --- a/libcacard/vreader.c
> +++ b/libcacard/vreader.c
> @@ -10,7 +10,9 @@
>  #endif
>  #define G_LOG_DOMAIN "libcacard"
>  
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>  
>  #include "vcard.h"
>  #include "vcard_emul.h"
> diff --git a/libcacard/vscclient.c b/libcacard/vscclient.c
> index fa6041d..0652684 100644
> --- a/libcacard/vscclient.c
> +++ b/libcacard/vscclient.c
> @@ -10,14 +10,20 @@
>   * See the COPYING.LIB file in the top-level directory.
>   */
>  
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <string.h>
>  #ifndef _WIN32
>  #include <sys/socket.h>
>  #include <netinet/in.h>
>  #include <netdb.h>
> +#include <unistd.h>
>  #define closesocket(x) close(x)
> +#else
> +#include <getopt.h>
>  #endif
>  
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>  
>  #include "vscard_common.h"
>  
>