also renamed the parser array itself. 'fun' is not very descriptive

Author: badasahog

src/fread.c

@@ -1206,13 +1206,13 @@ static void parse_bool_yn(FieldParseContext *ctx)
 
 /* How to register a new parser
  *  (1) Write the parser
- *  (2) Add it to fun array here
+ *  (2) Add it to parser_funs array here
  *  (3) Extend disabled_parsers, typeName, and typeSize here as appropriate
  *  (4) Extend colType typdef in fread.h as appropriate
  *  (5) Extend typeSxp, typeRName, typeEnum in freadR.c as appropriate
  */
 typedef void (*reader_fun_t)(FieldParseContext *ctx);
-static const reader_fun_t fun[NUMTYPE] = {
+static const reader_fun_t parser_funs[NUMTYPE] = {
   &Field,        // CT_DROP
   &parse_empty,  // CT_EMPTY
   &parse_bool_10,
@@ -1260,11 +1260,11 @@ static int detect_types(const char **pch, int ncol, bool *bumped)
         dec = '.';
       }
 
-      fun[tmpType[field]](&fctx);
+      parser_funs[tmpType[field]](&fctx);
       if (end_of_field(ch)) break;
       skip_white(&ch);
       if (end_of_field(ch)) break;
-      ch = end_NA_string(fieldStart);  // fieldStart here is correct (already after skip_white above); fun[]() may have part processed field so not ch
+      ch = end_NA_string(fieldStart);  // fieldStart here is correct (already after skip_white above); parser_funs[]() may have part processed field so not ch
       skip_white(&ch);
       if (end_of_field(ch)) break;
       ch = fieldStart;
@@ -1275,7 +1275,7 @@ static int detect_types(const char **pch, int ncol, bool *bumped)
       }
       if (*ch == quote && quote) {  // && quote to exclude quote='\0' (user passed quote="")
         ch++;
-        fun[tmpType[field]](&fctx);
+        parser_funs[tmpType[field]](&fctx);
         if (*ch == quote) {
           ch++;
           skip_white(&ch);
@@ -2459,7 +2459,7 @@ int freadMain(freadMainArgs _args)
               // DTPRINT(_("Field %d: '%.10s' as type %d  (tch=%p)\n"), j+1, tch, type[j], tch);
               fieldStart = tch;
               int8_t thisType = type[j];  // fetch shared type once. Cannot read half-written byte is one reason type's type is single byte to avoid atomic read here.
-              fun[IGNORE_BUMP(thisType)](&fctx);
+              parser_funs[IGNORE_BUMP(thisType)](&fctx);
               if (*tch != sep) break;
               int8_t thisSize = size[j];
               if (thisSize) ((char**) targets)[thisSize] += thisSize;  // 'if' for when rereading to avoid undefined NULL+0
@@ -2523,7 +2523,7 @@ int freadMain(freadMainArgs _args)
                 if (!end_of_field(tch)) tch = afterSpace; // else it is the field_end, we're on closing sep|eol and we'll let processor write appropriate NA as if field was empty
                 if (*tch == quote && quote) { quoted = true; tch++; }
               } // else Field() handles NA inside it unlike other processors e.g. ,, is interpreted as "" or NA depending on option read inside Field()
-              fun[IGNORE_BUMP(thisType)](&fctx);
+              parser_funs[IGNORE_BUMP(thisType)](&fctx);
 
               bool typeBump = false;
               if (quoted) {   // quoted was only set to true with '&& quote' above (=> quote!='\0' now)