Message ID | 5367A4FA.4040000@redhat.com |
---|---|
State | New |
Headers | show |
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 <alevy@redhat.com> > Tested-by: Alon Levy <alevy@redhat.com> 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 <alevy@redhat.com> > 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
On 05/05/2014 07:36 PM, Michael Tokarev wrote: > 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 <alevy@redhat.com> >> Tested-by: Alon Levy <alevy@redhat.com> > > 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. I've tested and reviewed 7191b2c43eecc52994924245720c534ea1a0dc84 so v2, my bad. > >> 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 <alevy@redhat.com> >> 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 >
diff --git a/libcacard/vreader.c b/libcacard/vreader.c index 91799b4..d1e05af 100644 --- a/libcacard/vreader.c +++ b/libcacard/vreader.c @@ -272,12 +272,12 @@ vreader_xfr_bytes(VReader *reader, response = vcard_make_response(status); card_status = VCARD_DONE; } else { - g_debug("%s: CLS=0x%x,INS=0x%x,P1=0x%x,P2=0x%x,Lc=%d,Le=%d %s\n", + g_debug("%s: CLS=0x%x,INS=0x%x,P1=0x%x,P2=0x%x,Lc=%d,Le=%d %s", __func__, apdu->a_cla, apdu->a_ins, apdu->a_p1, apdu->a_p2, apdu->a_Lc, apdu->a_Le, apdu_ins_to_string(apdu->a_ins)); card_status = vcard_process_apdu(card, apdu, &response); if (response) { - g_debug("%s: status=%d sw1=0x%x sw2=0x%x len=%d (total=%d)\n", + g_debug("%s: status=%d sw1=0x%x sw2=0x%x len=%d (total=%d)", __func__, response->b_status, response->b_sw1, response->b_sw2, response->b_len, response->b_total_len); }