dexter submitted this change.

View Change

Approvals: laforge: Looks good to me, approved jolly: Looks good to me, but someone else must approve Jenkins Builder: Verified
esipa_asn1_handler_utils: use maps:find

We can replace maps:is_key and maps:get with a single maps:find call.

Change-Id: I6cc151deef161f3ca4726e6b81ae1f5cc80ad184
Related: SYS#8100
---
M src/esipa_asn1_handler_utils.erl
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/esipa_asn1_handler_utils.erl b/src/esipa_asn1_handler_utils.erl
index ac7d6f0..2d537fd 100644
--- a/src/esipa_asn1_handler_utils.erl
+++ b/src/esipa_asn1_handler_utils.erl
@@ -26,9 +26,9 @@

process_euiccPackageResult(Pid, EuiccPackageResult, EsipaReq, TransactionId) ->
WorkBind = fun(Map) ->
- case maps:is_key(eimTransactionId, Map) of
- true ->
- mnesia_db_work:bind(Pid, maps:get(eimTransactionId, Map));
+ case maps:find(eimTransactionId, Map) of
+ {ok, Value} ->
+ mnesia_db_work:bind(Pid, Value);
_ ->
ok
end

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

Gerrit-MessageType: merged
Gerrit-Project: onomondo-eim
Gerrit-Branch: master
Gerrit-Change-Id: I6cc151deef161f3ca4726e6b81ae1f5cc80ad184
Gerrit-Change-Number: 42959
Gerrit-PatchSet: 9
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: jolly <andreas@eversberg.eu>
Gerrit-Reviewer: laforge <laforge@osmocom.org>