dexter submitted this change.

View Change

Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve
pySim/runtime: use log.warning instead of log.warn

The python logger method warn is deprecated since pyton 3.3, let's us
the warning method as suggested.

Change-Id: I3a4c0ca43768198ac6011ebe79050f91c04862e5
---
M pySim/runtime.py
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/pySim/runtime.py b/pySim/runtime.py
index 19d3538..54f431b 100644
--- a/pySim/runtime.py
+++ b/pySim/runtime.py
@@ -116,7 +116,7 @@
for a in aids_unknown:
log.info(" unknown: %s (EF.DIR)" % a)
else:
- log.warn("EF.DIR seems to be empty!")
+ log.warning("EF.DIR seems to be empty!")

# Some card applications may not be registered in EF.DIR, we will actively
# probe for those applications
@@ -557,8 +557,8 @@
raise TypeError("Data length (%u) exceeds %s size (%u) by %u bytes" %
(data_len, writeable_name, writeable_size, data_len - writeable_size))
elif data_len < writeable_size:
- log.warn("Data length (%u) less than %s size (%u), leaving %u unwritten bytes at the end of the %s" %
- (data_len, writeable_name, writeable_size, writeable_size - data_len, writeable_name))
+ log.warning("Data length (%u) less than %s size (%u), leaving %u unwritten bytes at the end of the %s" %
+ (data_len, writeable_name, writeable_size, writeable_size - data_len, writeable_name))

def update_binary(self, data_hex: str, offset: int = 0):
"""Update transparent EF binary data.

To view, visit change 41766. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I3a4c0ca43768198ac6011ebe79050f91c04862e5
Gerrit-Change-Number: 41766
Gerrit-PatchSet: 3
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>