[Yt-svn] yt-commit r698 - branches/parallel_profiles/yt/lagos

[mturk at wrangler.dreamhost.com]

Author: mturk
Date: Sat Jul 26 16:13:47 2008
New Revision: 698
URL: http://yt.spacepope.org/changeset/698

Log:
I think this did it; leaving the files open seems to have been the issue.
Oops.



Modified:
   branches/parallel_profiles/yt/lagos/HDF5LightReader.c

Modified: branches/parallel_profiles/yt/lagos/HDF5LightReader.c
==============================================================================
--- branches/parallel_profiles/yt/lagos/HDF5LightReader.c	(original)
+++ branches/parallel_profiles/yt/lagos/HDF5LightReader.c	Sat Jul 26 16:13:47 2008
@@ -476,6 +476,11 @@
     Py_ssize_t num_sets = 0;
     Py_ssize_t num_grids = 0;
 
+    if (!PyArg_ParseTuple(args, "sOO",
+            &filename, &grid_names, &set_names))
+        return PyErr_Format(_hdf5ReadError,
+               "ReadMultipleGrids: Invalid parameters.");
+
     num_grids = PyList_Size(grid_names);
     num_sets = PyList_Size(set_names);
     PyObject *grids_dict = PyDict_New(); // New reference
@@ -488,11 +493,6 @@
     file_id = grid_node = 0;
     int i, n;
 
-    if (!PyArg_ParseTuple(args, "sOO",
-            &filename, &grid_names, &set_names))
-        return PyErr_Format(_hdf5ReadError,
-               "ReadMultipleGrids: Invalid parameters.");
-
     file_id = H5Fopen (filename, H5F_ACC_RDONLY, H5P_DEFAULT); 
 
     if (file_id < 0) {
@@ -531,6 +531,7 @@
         H5Gclose(grid_node);
     }
 
+    H5Fclose(file_id);
     PyObject *return_value = Py_BuildValue("N", grids_dict);
     return return_value;