From 9ddd235a2c54b989e0627552e14f98c484cbbedd Mon Sep 17 00:00:00 2001 From: Philipp Maier Date: Wed, 27 May 2026 14:59:23 +0200 Subject: [PATCH] pySim/ts_51_011: rewrite comment for better understanding The comment reads like that we were applying TS 102.221 here, but we only mean our internal decoding format. The spec that actually matters here is TS 51.011. Let's rephrase the comment so that this becomes more clear. Related: OS#7018 Change-Id: Ie0184eea25f4d9f4baf9ab137c53a926edba2bf8 --- pySim/ts_51_011.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/pySim/ts_51_011.py b/pySim/ts_51_011.py index 89ff8248..4cad0490 100644 --- a/pySim/ts_51_011.py +++ b/pySim/ts_51_011.py @@ -1263,9 +1263,11 @@ class CardProfileSIM(CardProfile): @staticmethod def decode_select_response(resp_hex: str) -> object: - # we try to build something that resembles a dict resulting from the TLV decoder - # of TS 102.221 (FcpTemplate), so that higher-level code only has to deal with one - # format of SELECT response + """ + Decode the select response to a dict representation, similar to the one of TS 102.221 (see ts_102_221.py, + class FcpTemplate), so that higher-level code only has to deal with one respresentation. See also + 3GPP TS 51.011, section 9.2.1 + """ resp_bin = h2b(resp_hex) struct_of_file_map = { 0: 'transparent',