From 4b0a79126af20099588865b69362f4d73d2a93a0 Mon Sep 17 00:00:00 2001
From: Mike Taves <mwtoews@gmail.com>
Date: Fri, 26 Jul 2019 11:05:19 +1200
Subject: [PATCH] refactor whitespace in NAM file and MFList output files

---
 flopy/discretization/grid.py | 11 +++++------
 flopy/mbase.py               | 11 +++++++----
 flopy/modflow/mf.py          |  2 +-
 flopy/utils/util_list.py     | 16 +++++++++-------
 4 files changed, 22 insertions(+), 18 deletions(-)

diff --git a/flopy/discretization/grid.py b/flopy/discretization/grid.py
index 05bad947ce..9ca8bcadb6 100644
--- a/flopy/discretization/grid.py
+++ b/flopy/discretization/grid.py
@@ -162,12 +162,11 @@ def __init__(self, grid_type=None, top=None, botm=None, idomain=None,
     # access to basic grid properties
     ###################################
     def __repr__(self):
-        s = "xll:{0:<.10G}; yll:{1:<.10G}; rotation:{2:<G}; ". \
-            format(self.xoffset, self.yoffset, self.angrot)
-        s += "proj4_str:{0}; ".format(self.proj4)
-        s += "units:{0}; ".format(self.units)
-        s += "lenuni:{0}; ".format(self.lenuni)
-        return s
+        return (
+            "xll:{0:<.10G}; yll:{1:<.10G}; rotation:{2:<G}; proj4_str:{3}; "
+            "units:{4}; lenuni:{5}"
+            .format(self.xoffset, self.yoffset, self.angrot, self.proj4,
+                    self.units, self.lenuni))
 
     @property
     def grid_type(self):
diff --git a/flopy/mbase.py b/flopy/mbase.py
index 05ff34b295..bba4c53263 100644
--- a/flopy/mbase.py
+++ b/flopy/mbase.py
@@ -915,14 +915,17 @@ def get_name_file_entries(self):
         ----------
 
         """
-        s = ''
+        lines = []
         for p in self.packagelist:
             for i in range(len(p.name)):
                 if p.unit_number[i] == 0:
                     continue
-                s += '{:14s} {:5d}  '.format(p.name[i], p.unit_number[i]) + \
-                     '{:s} {:s}\n'.format(p.file_name[i], p.extra[i])
-        return s
+                s = '{:14s} {:5d}  {}'.format(
+                        p.name[i], p.unit_number[i], p.file_name[i])
+                if p.extra[i]:
+                    s += ' ' + p.extra[i]
+                lines.append(s)
+        return '\n'.join(lines) + '\n'
 
     def has_package(self, name):
         """
diff --git a/flopy/modflow/mf.py b/flopy/modflow/mf.py
index a930539565..d10f377dc6 100644
--- a/flopy/modflow/mf.py
+++ b/flopy/modflow/mf.py
@@ -414,7 +414,7 @@ def write_name_file(self):
         f_nam.write('{}\n'.format(self.heading))
         if self.structured:
             f_nam.write('#' + str(self.modelgrid))
-        f_nam.write(" ;start_datetime:{0}\n".format(self.start_datetime))
+        f_nam.write("; start_datetime:{0}\n".format(self.start_datetime))
         if self.version == 'mf2k':
             if self.glo.unit_number[0] > 0:
                 f_nam.write('{:14s} {:5d}  {}\n'.format(self.glo.name[0],
diff --git a/flopy/utils/util_list.py b/flopy/utils/util_list.py
index ce7dbfd6d3..d79c653b35 100644
--- a/flopy/utils/util_list.py
+++ b/flopy/utils/util_list.py
@@ -695,13 +695,15 @@ def write_transient(self, f, single_per=None, forceInternal=False):
 
             if kper_vtype == np.recarray:
                 name = f.name
-                f.close()
-                f = open(name, 'ab+')
-                # print(f)
-                self.__tofile(f, kper_data)
-                f.close()
-                f = open(name, 'a')
-                # print(f)
+                if self.__binary:
+                    f.close()
+                    # switch file append mode to binary
+                    with open(name, 'ab+') as f:
+                        self.__tofile(f, kper_data)
+                    # continue back to non-binary
+                    f = open(name, 'a')
+                else:
+                    self.__tofile(f, kper_data)
             elif kper_vtype == str:
                 f.write('         open/close ' + kper_data)
                 if self.__binary: