changeset 1473:c543f5ae92d5

Cleanup dmReadDataFile() helper function.
author Matti Hamalainen <ccr@tnsp.org>
date Fri, 11 May 2018 01:59:37 +0300
parents b9d3577d8290
children 6b1f41ca300a
files src/dmfile.c src/dmfile.h
diffstat 2 files changed, 14 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/src/dmfile.c	Thu May 10 23:08:09 2018 +0300
+++ b/src/dmfile.c	Fri May 11 01:59:37 2018 +0300
@@ -61,25 +61,26 @@
 #define BUF_SIZE_INITIAL   (16*1024)
 #define BUF_SIZE_GROW      (4*1024)
 
-int dmReadDataFile(FILE *inFile, const char *filename, Uint8 **pbuf, size_t *pbufSize)
+int dmReadDataFile(FILE *ffh, const char *filename, Uint8 **pbuf, size_t *pbufSize)
 {
-    FILE *f;
+    FILE *fh = NULL;
     int res = DMERR_OK;
     Uint8 *dataBuf = NULL;
     size_t readSize, dataSize, dataRead, dataPos;
 
-    if (inFile != NULL)
-        f = inFile;
+    if (ffh != NULL)
+        fh = ffh;
     else
     if (filename != NULL)
     {
-        if ((f = fopen(filename, "rb")) == NULL)
+        if ((fh = fopen(filename, "rb")) == NULL)
         {
             return dmError(DMERR_FOPEN,
                 "Could not open '%s' for reading.\n", filename);
         }
     }
-    else
+
+    if (fh == NULL)
     {
         return dmError(DMERR_NULLPTR,
             "NULL filename and stream pointers.\n");
@@ -94,12 +95,11 @@
         goto error;
     }
 
-    dataPos = 0;
-    dataRead = 0;
-
-    while (!feof(f) && !ferror(f))
+    // Read the data
+    dataPos = dataRead = 0;
+    while (!feof(fh) && !ferror(fh))
     {
-        size_t read = fread(dataBuf + dataPos, 1, readSize, f);
+        size_t read = fread(dataBuf + dataPos, 1, readSize, fh);
         dataPos += read;
         dataRead += read;
 
@@ -123,8 +123,8 @@
     *pbuf = dataBuf;
 
 error:
-    if (f != inFile)
-        fclose(f);
+    if (ffh != fh)
+        fclose(fh);
 
     return res;
 }
--- a/src/dmfile.h	Thu May 10 23:08:09 2018 +0300
+++ b/src/dmfile.h	Fri May 11 01:59:37 2018 +0300
@@ -31,7 +31,7 @@
 BOOL    dm_fwrite_byte(FILE *f, const Uint8);
 
 
-int     dmReadDataFile(FILE *inFile, const char *filename, Uint8 **buf, size_t *size);
+int     dmReadDataFile(FILE *inFile, const char *filename, Uint8 **pbuf, size_t *pbufSize);
 
 
 #ifdef __cplusplus