Revert change that was added by kvaragan due to a bug that was fixed. See comment thread for PR 150.

jlgreathouse · jlgreathouse · commit 0e0b8e54d62b · 2015-10-14T21:23:39.000-05:00
diff --git a/src/library/io/mm-reader.cpp b/src/library/io/mm-reader.cpp
@@ -568,7 +568,6 @@ clsparseSCsrMatrixfromFile(clsparseCsrMatrix* csrMatx, const char* filePath, cls
     MatrixMarketReader< cl_float > mm_reader;
     if( mm_reader.MMReadFormat( filePath ) )
         return clsparseInvalidFile;
-#if 0
     // BUG: We need to check to see if openCL buffers currently exist and deallocate them first!
     // FIX: Below code will check whether the buffers were allocated in the first place;
     {
@@ -590,7 +589,6 @@ clsparseSCsrMatrixfromFile(clsparseCsrMatrix* csrMatx, const char* filePath, cls
         if (validationStatus != clsparseSuccess)
             return validationStatus;
     }
-#endif
 
     // JPA: Shouldn't that just be an assertion check? It seems to me that
     // the user have to call clsparseHeaderfromFile before calling this function,

Original file line number	Diff line number	Diff line change
`@@ -568,7 +568,6 @@ clsparseSCsrMatrixfromFile(clsparseCsrMatrix* csrMatx, const char* filePath, cls`
`568`	`568`	`MatrixMarketReader< cl_float > mm_reader;`
`569`	`569`	`if( mm_reader.MMReadFormat( filePath ) )`
`570`	`570`	`return clsparseInvalidFile;`
`571`		`-#if 0`
`572`	`571`	`// BUG: We need to check to see if openCL buffers currently exist and deallocate them first!`
`573`	`572`	`// FIX: Below code will check whether the buffers were allocated in the first place;`
`574`	`573`	`{`
`@@ -590,7 +589,6 @@ clsparseSCsrMatrixfromFile(clsparseCsrMatrix* csrMatx, const char* filePath, cls`
`590`	`589`	`if (validationStatus != clsparseSuccess)`
`591`	`590`	`return validationStatus;`
`592`	`591`	`}`
`593`		`-#endif`
`594`	`592`
`595`	`593`	`// JPA: Shouldn't that just be an assertion check? It seems to me that`
`596`	`594`	`// the user have to call clsparseHeaderfromFile before calling this function,`