From 0f96c0281511f4b21a5a3a31e906c13e6c8982c9 Mon Sep 17 00:00:00 2001 From: Harald Welte Date: Sun, 23 May 2021 14:07:23 +0200 Subject: [PATCH] commands: remove superfluous getter/setter for cla_byte property There's little point in having a getter+setter for a property if all it does is assigning a value to an attribute of self. That works without any property methods Change-Id: Id214cc83a29e8aa88f4e1413e07b419285c1b7ff --- pySim/commands.py | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/pySim/commands.py b/pySim/commands.py index 800c2dd6..689cbeab 100644 --- a/pySim/commands.py +++ b/pySim/commands.py @@ -29,7 +29,7 @@ from pySim.exceptions import SwMatchError class SimCardCommands(object): def __init__(self, transport): self._tp = transport - self._cla_byte = "a0" + self.cla_byte = "a0" self.sel_ctrl = "0000" # Extract a single FCP item from TLV @@ -89,13 +89,6 @@ class SimCardCommands(object): """Return the ATR of the currently inserted card.""" return self._tp.get_atr() - @property - def cla_byte(self): - return self._cla_byte - @cla_byte.setter - def cla_byte(self, value): - self._cla_byte = value - def try_select_path(self, dir_list): """ Try to select a specified path given as list of hex-string FIDs""" rv = []