From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=60178 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Q4EKX-0004np-9Z for qemu-devel@nongnu.org; Mon, 28 Mar 2011 11:26:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Q4EKV-0000Jq-Vo for qemu-devel@nongnu.org; Mon, 28 Mar 2011 11:26:08 -0400 Received: from mx1.redhat.com ([209.132.183.28]:20115) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Q4EKV-0000Jh-Kx for qemu-devel@nongnu.org; Mon, 28 Mar 2011 11:26:07 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p2SFQ6sY006748 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Mon, 28 Mar 2011 11:26:06 -0400 Received: from [10.3.113.75] (ovpn-113-75.phx2.redhat.com [10.3.113.75]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id p2SFQ5o8000597 for ; Mon, 28 Mar 2011 11:26:06 -0400 Message-ID: <4D90A879.2030600@redhat.com> Date: Mon, 28 Mar 2011 17:25:45 +0200 From: Jes Sorensen MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH v23 08/11] libcacard: add passthru References: <1300886393-2799-1-git-send-email-alevy@redhat.com> <1300886393-2799-9-git-send-email-alevy@redhat.com> <4D908CD5.5030301@redhat.com> <20110328152146.GE2741@playa.redhat.com> In-Reply-To: <20110328152146.GE2741@playa.redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org On 03/28/11 17:21, Alon Levy wrote: >> > Where does this odd LONG type come from? I think Windows uses LONG so >> > having your own type would conflict with that, and of course we don't >> > really want Windows types directly in QEMU either, so I am curious? >> > > This comes from pcsclite (which means I need to add the dependency to configure, > I'll fix this). From there it comes via an include called wintypes.h. the PCSC API > is actually a copy of a microsoft API, so it makes sense that it uses those types. > > On windows pcsclite won't be needed, since it has a native equivalent API, so there > won't be a redefinition of LONG. > > So the real fix is that I need to add yet another dependency for qemu's configure, > for pcsc-lite - but right now the whole libcacard/passthru.c is protected by an > "#ifdef USE_PASSTHRU", which we don't define - so in effect this is currently dead > code. I could even just remove this commit in its entirety atm, maybe best? Notice > that has nothing to do with ccid-card-emulated or with ccid-card-passthru. I will > need it later, but I need to add the pcsclite configure check anyway. So is this ok, > I'll drop this patch? > Interesting - yeah if the code isn't necessary, lets drop it for now. It will be easier to get this in now, and the extra bits later when they are ready. Cheers, Jes