laforge submitted this change.

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved
  msuraev: Looks good to me, but someone else must approve

pySim-prog: rename write_parameters function.

The function name "write_parameters" is very generic and since it is
called during the programming cycle it should be made clear that it is
not about writing parameters to the card.

Change-Id: Idaba672987230d7d0dd500409f9fe0b94ba39370
---
M pySim-prog.py
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/pySim-prog.py b/pySim-prog.py
index dbf1148..f106018 100755
--- a/pySim-prog.py
+++ b/pySim-prog.py
@@ -669,7 +669,7 @@
         conn.close()
 
 
-def write_parameters(opts, params):
+def write_parameters_to_csv_and_hlr(opts, params):
     write_params_csv(opts, params)
     write_params_hlr(opts, params)
 
@@ -764,8 +764,8 @@
     else:
         print("Dry Run: NOT PROGRAMMING!")
 
-    # Write parameters permanently
-    write_parameters(opts, cp)
+    # Write parameters to a specified CSV file or an HLR database (not the card)
+    write_parameters_to_csv_and_hlr(opts, cp)
 
     # Batch mode state update and save
     if opts.num is not None:

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