home · contact · privacy
Use not f_name variable but __func__, standardize function name writing.
[plomrogue] / src / common / readwrite.c
index c9e22d73d3ae16dd7fd37a19b17908cebc07921e..f05c1285a9d5cdda58ee909d4814aea019318b03 100644 (file)
@@ -21,29 +21,27 @@ static char * build_temp_path(char * path);
 
 static char * build_temp_path(char * path)
 {
-    char * f_name = "build_temp_path";
     char * suffix_tmp = "_tmp";
     uint16_t size = strlen(path) + strlen(suffix_tmp) + 1;
-    char * path_tmp = try_malloc(size, f_name);
+    char * path_tmp = try_malloc(size, __func__);
     int test = sprintf(path_tmp, "%s%s", path, suffix_tmp);
-    exit_trouble(test < 0, f_name, "sprintf()");
+    exit_trouble(test < 0, __func__, "sprintf");
     return path_tmp;
 }
 
 
 
-extern FILE * try_fopen(char * path, char * mode, char * f)
+extern FILE * try_fopen(char * path, char * mode, const char * f)
 {
-    char * f_name = "try_fopen()";
     char * msg1 = "Trouble in ";
-    char * msg2 = " with fopen() (mode '";
+    char * msg2 = " with fopen (mode '";
     char * msg3 = "') on path '";
     char * msg4 = "'.";
     uint16_t size = strlen(msg1) + strlen(msg2) + strlen(msg3) + strlen(msg4)
                     + strlen(f) + strlen(path) + strlen(mode) + 1;
-    char * msg = try_malloc(size, f_name);
+    char * msg = try_malloc(size, __func__);
     int test = sprintf(msg, "%s%s%s%s%s%s%s", msg1,f,msg2,mode,msg3,path,msg4);
-    exit_trouble(test < 0, f_name, "sprintf()");
+    exit_trouble(test < 0, __func__, "sprintf");
     FILE * file_p = fopen(path, mode);
     exit_err(NULL == file_p, msg);
     free(msg);
@@ -52,42 +50,42 @@ extern FILE * try_fopen(char * path, char * mode, char * f)
 
 
 
-extern void try_fclose(FILE * file, char * f)
+extern void try_fclose(FILE * file, const char * f)
 {
-    exit_trouble(fclose(file), f, "fclose()");
+    exit_trouble(fclose(file), f, "fclose");
 }
 
 
 
 extern void try_fwrite(void * ptr, size_t size, size_t nmemb, FILE * stream,
-                       char * f)
+                       const char * f)
 {
-    exit_trouble(0 == fwrite(ptr, size, nmemb, stream), f, "fwrite()");
+    exit_trouble(0 == fwrite(ptr, size, nmemb, stream), f, "fwrite");
 }
 
 
 
-extern void try_fputc(uint8_t c, FILE * file, char * f)
+extern void try_fputc(uint8_t c, FILE * file, const char * f)
 {
-    exit_trouble(EOF == fputc(c, file), f, "fputc()");
+    exit_trouble(EOF == fputc(c, file), f, "fputc");
 }
 
 
 
-extern int try_fgetc(FILE * file, char * f)
+extern int try_fgetc(FILE * file, const char * f)
 {
     clearerr(file); /* OSX' (BSD?) fgetc() needs this to undo previous EOFs. */
     int test = fgetc(file);
-    exit_trouble(EOF == test && ferror(file), f, "fgetc()");
+    exit_trouble(EOF == test && ferror(file), f, "fgetc");
     return test;
 }
 
 
 
-extern char * try_fgets(char * line, int linemax, FILE * file, char * f)
+extern char * try_fgets(char * line, int linemax, FILE * file, const char * f)
 {
     char * test = fgets(line, linemax, file);
-    exit_trouble(NULL == test && ferror(file), f, "fgets()");
+    exit_trouble(NULL == test && ferror(file), f, "fgets");
     return test;
 }
 
@@ -95,39 +93,37 @@ extern char * try_fgets(char * line, int linemax, FILE * file, char * f)
 
 extern FILE * atomic_write_start(char * path, char ** path_tmp)
 {
-    char * f_name = "atomic_write_start()";
     *path_tmp = build_temp_path(path);
-    return try_fopen(*path_tmp, "w", f_name);
+    return try_fopen(*path_tmp, "w", __func__);
 }
 
 
 
 extern void atomic_write_finish(FILE * file, char * path, char * path_tmp)
 {
-    char * f_name = "atomic_write_finish()";
-    try_fclose(file, f_name);
+    try_fclose(file, __func__);
     char * msg1 = "Trouble in ";
     char * msg4 = "'.";
     if (!access(path, F_OK))
     {
-        char * msg2 = " with unlink() on path '";
+        char * msg2 = " with unlink on path '";
         uint16_t size = strlen(msg1) + strlen(msg2) + strlen(msg4)
-                        + strlen(f_name) + strlen(path) + 1;
-        char * msg = try_malloc(size, f_name);
-        int test = sprintf(msg, "%s%s%s%s%s", msg1, f_name, msg2, path, msg4);
-        exit_trouble(test < 0, f_name, "sprintf()");
+                        + strlen(__func__) + strlen(path) + 1;
+        char * msg = try_malloc(size, __func__);
+        int test = sprintf(msg, "%s%s%s%s%s", msg1, __func__, msg2, path, msg4);
+        exit_trouble(test < 0, __func__, "sprintf");
         exit_err(unlink(path), msg);
         free(msg);
     }
-    char * msg2 = " with rename() from '";
+    char * msg2 = " with rename from '";
     char * msg3 = "' to '";
-    uint16_t size =   strlen(msg1) + strlen(f_name) + strlen(msg2) +
+    uint16_t size =   strlen(msg1) + strlen(__func__) + strlen(msg2) +
                     + strlen(path_tmp) + strlen(msg3) + strlen(path)
                     + strlen(msg4) + 1;
-    char * msg = try_malloc(size, f_name);
+    char * msg = try_malloc(size, __func__);
     int test = sprintf(msg, "%s%s%s%s%s%s%s",
-                            msg1, f_name, msg2, path_tmp, msg3, path, msg4);
-    exit_trouble(test < 0, f_name, "sprintf()");
+                            msg1, __func__, msg2, path_tmp, msg3, path, msg4);
+    exit_trouble(test < 0, __func__, "sprintf");
     exit_err(rename(path_tmp, path), msg);
     free(msg);
     free(path_tmp);
@@ -137,7 +133,6 @@ extern void atomic_write_finish(FILE * file, char * path, char * path_tmp)
 
 extern void detect_atomic_leftover(char * path)
 {
-    char * f_name = "detect_atomic_leftover()";
     char * path_tmp = build_temp_path(path);
     char * part1 = "Found file '";
     char * part2 = "' that may be a leftover from an aborted previous attempt "
@@ -145,9 +140,9 @@ extern void detect_atomic_leftover(char * path)
     char * part3 = "'. Aborting until the matter is solved by (re-)moving it.";
     uint32_t size =   strlen(part1) + strlen(path_tmp) + strlen(part2)
                     + strlen(path) + strlen(part3) + 1;
-    char * msg = try_malloc(size, f_name);
+    char * msg = try_malloc(size, __func__);
     int test = sprintf(msg, "%s%s%s%s%s", part1, path_tmp, part2, path, part3);
-    exit_trouble(test < 0, f_name, "sprintf()");
+    exit_trouble(test < 0, __func__, "sprintf");
     exit_err(!access(path_tmp, F_OK), msg);
     free(msg);
     free(path_tmp);
@@ -157,19 +152,18 @@ extern void detect_atomic_leftover(char * path)
 
 extern uint32_t textfile_width(FILE * file)
 {
-    char * f_name = "textfile_width()";
     int c = 0;
     uint32_t c_count = 0;
     uint32_t linemax = 0;
     while (1)
     {
-        c = try_fgetc(file, f_name);
+        c = try_fgetc(file, __func__);
         if (EOF == c)
         {
             break;
         }
         c_count++;
-        exit_trouble(UINT32_MAX == c_count, f_name, "too large text file line");
+        exit_trouble(UINT32_MAX==c_count, __func__, "too large text file line");
         if ('\n' == c)
         {
             if (c_count > linemax)
@@ -183,6 +177,6 @@ extern uint32_t textfile_width(FILE * file)
   {                                /* line / lack newline chars.            */
       linemax = c_count;
    }
-   exit_trouble(-1 == fseek(file, 0, SEEK_SET), f_name, "fseek()");
+   exit_trouble(-1 == fseek(file, 0, SEEK_SET), __func__, "fseek");
    return linemax;
 }