From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54136) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WhLs7-00023O-OP for qemu-devel@nongnu.org; Mon, 05 May 2014 12:36:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WhLs3-0005VK-CL for qemu-devel@nongnu.org; Mon, 05 May 2014 12:36:07 -0400 Received: from isrv.corpit.ru ([86.62.121.231]:57115) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WhLs3-0005Uy-4q for qemu-devel@nongnu.org; Mon, 05 May 2014 12:36:03 -0400 Message-ID: <5367BDF1.3020201@msgid.tls.msk.ru> Date: Mon, 05 May 2014 20:36:01 +0400 From: Michael Tokarev MIME-Version: 1.0 References: <1398751349-20869-1-git-send-email-mjt@msgid.tls.msk.ru> <5367A4FA.4040000@redhat.com> In-Reply-To: <5367A4FA.4040000@redhat.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/5] glib thread interface and libcacard cleanups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alon Levy , qemu-devel 05.05.2014 18:49, Alon Levy wrote: > On 04/29/2014 09:02 AM, Michael Tokarev wrote: >> Basically libgthread has been rewritten in glib version 2.31, and old ways >> to use thread primitives stopped working while new ways appeared. The two >> interfaces were sufficiently different to warrant large ifdeffery across all >> code using it. [...] [] > Reviewed-by: Alon Levy > Tested-by: Alon Levy Hmm. Now I'm a bit confused. Which version did you test? :) I posted a v2 patch which splits one of the changes into two (pstrcpy to memcpy conversion in libcacard), added some more (minor) changes (which should not affect libcacard code in any way), and adjusted commit messages. The main code is not affected (or should not be), so Tested-by probably may stay, except of the pstrcpy to memcpy patch (http://patchwork.ozlabs.org/patch/345002/) which may affect libcacard. Here's the v2: http://lists.gnu.org/archive/html/qemu-devel/2014-05/msg00286.html If you tested the git branch which I referred to, that's the v2, not original submission which you're replying to. > This would be nice to have too (it has nothing to do with your patchset, > but it would save me a pull request): > > commit 2fc95f8bc1912e2de243389d9d102a5a28267f31 > Author: Alon Levy > Date: Mon May 5 17:41:32 2014 +0300 > > libcacard: remove unnecessary EOL from debug prints Well, this can easily go to -trivial, as I'm planning to send a pull request for it soon anyway. Thank you! /mjt