code refactor

related to #8 (closed):

• I kept changes per commit small, so the number of commits got kind of numerous...

src/icartt/dataset.py

@@ -125,7 +125,7 @@ class DataStore2110(collections.UserDict):
         self.add(**newdata)
 
     def addBulkDep(self, ivar, raw):
-        nlines, nvars = raw.shape
+        nlines, nvars = raw.shape # nvars not used
         self._addDeplines(ivar, raw, nlines)
 
     def _addDeplines(self, ivar, raw, n):
@@ -491,6 +491,10 @@ class Dataset:
             self.format = Formats(int(dmp[1]))
         except:
             raise ValueError("ICARTT format {:d} not implemented".format(dmp[1]))
+            # TODO except clause could be re-written like
+            # except ValueError as ve:
+            #     raise ValueError(f"ICARTT format {dmp[1]:d} not implemented") from ve
+            # to show the error trace
 
         if len(dmp) > 2:
             self.version = dmp[2]
@@ -926,6 +930,8 @@ class Dataset:
             else:
                 self.inputFhandle = f
 
+        # TODO should we add a filename characters check, as we have it for the variable names?
+
             self.readHeader(splitChar)
             if loadData:
                 self.endDefineMode()