Wireshark-bugs: [Wireshark-bugs] [Bug 7715] Bluetooth: Add SAP dissector
Date: Wed, 26 Sep 2012 01:03:09 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7715

--- Comment #23 from Michal Labedzki <michal.labedzki@xxxxxxxxx> 2012-09-26 01:03:07 PDT ---
(In reply to comment #22)
> Comment on attachment 9133 [details]
> [PATCH 2/3] iso7816: Add subdissector ISO 7816-3
> 
> Hi,
> 
> I committed a slightly different version of this in r45137.
> 
> The idea is to get a clearer separation between iso7816 atr and the rest.
> 
> I'm reluctant to using data as you proposed. As I didn't put anything useful
> into the info column until now, I just took what you proposed ;-)
> 
> Hope that's ok for you.
> 
> Regards,
> 
>    Martin

Hi, 
Thanks. However I only use boolean data parameter to avoid inconsistent "info
column". For now that looks:
"Sent TRANSFER_ATR_RESP - ResultCode: OK, request processed correctly - ATR: ,
ATR, TA(1)=True TB(1)=True TC(1)=False..."

What I want is:
"Sent TRANSFER_ATR_RESP - ResultCode: OK, request processed correctly - ATR:
TA(1)=True TB(1)=True TC(1)=False..."

But this is not very important for me. If I understand I should use structure
for data parameter and introduce header file (*.h) then will be ok, right?

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.