Changeset 3369 for trunk/Ohana/src/libfits/header/F_scan.c

Timestamp:

Mar 2, 2005, 6:02:44 PM (21 years ago)

Author:

eugene

Message:

cleaned handling of single quotes

File:

• trunk/Ohana/src/libfits/header/F_scan.c (modified) (4 diffs)

trunk/Ohana/src/libfits/header/F_scan.c

@@ -1 +1 @@
 # include "fits.h"
+
+char *fits_keyword_start (char *line);
+char *fits_keyword_end (char *line);
 
 int fits_scan (Header *header, char *field, char *mode, int N,...) {
@@ -14 +17 @@
 int fits_vscan (Header *header, char *field, char *mode, int N, va_list argp) {
 
-  char *p, *q, tmp[81];
+  char *p, *q, *s, tmp[81];
   int Nchar;
   double value;
@@ -25 +28 @@
   if (!strcmp (mode, "%S")) {
     strncpy (va_arg (argp, char *), p + 8, FT_HISTORY_LENGTH);
-    goto gotvalue;
+    return (TRUE);
   }
 
@@ -40 +43 @@
     fits_stripwhite (tmp);
     strcpy (va_arg (argp, char *), tmp);
-    goto gotvalue;
+    return (TRUE);
   }
 
   /* extract data into char array (exclude containing ' chars) */
   if (!strcmp (mode, "%s")) {
-    q = fits_keyword_end (p);
-    q -= 1;
-    p += 10;
+    s = fits_keyword_start (p); /* points at first char (not ') */
+    q = fits_keyword_end (p); /* points at following space or ' */
 
-    /* exclude containing ' chars, if they exist */
-    /* we are not requiring a string entry to have the quotes */
-    if (*p == 0x27) p++;
-    if (*q == 0x27) q--;
-
-    Nchar = MAX (0, (q - p + 1));
+    Nchar = MAX (0, (q - s));
     bzero (tmp, 81);
-    memcpy (tmp, p, Nchar);
+    memcpy (tmp, s, Nchar);
     fits_stripwhite (tmp);
     strcpy (va_arg (argp, char *), tmp);
-    goto gotvalue;
+    return (TRUE);
   }
   
   /* boolean data, requires int target */
   if (!strcmp (mode, "%t")) {
-    if (p[29]  == 'T') 
+    s = fits_keyword_start (p);
+    if (*s == 'T') {
       *va_arg (argp, int *) = TRUE;
-    if (p[29]  == 'F') 
+      return (TRUE);
+    }
+    if (*s == 'F') {
       *va_arg (argp, int *) = FALSE;   
-    goto gotvalue;
+      return (TRUE);
+    }
   }
 
   /* remaining options are numerical data */
   /* need to interpret 1.0d5 as 1.0e5 */
-  if (p[10] == 0x27) p++;  /** skip over initial ' (SOME BAD sources write floats with '') **/
-  value = strtod (p + 10, &q);
+  s = fits_keyword_start (p); /* points at first char (not ') */
+  q = fits_keyword_end (p); /* points at following space or ' */
+  value = strtod (s, &q);
   if ((*q == 'd') || (*q == 'D')) 
     value *= pow (10.0, atof (q + 1));
 
-  if (!strcmp (mode, "%d"))  { *va_arg (argp, int *)       = value; goto gotvalue; }
-  if (!strcmp (mode, "%u"))  { *va_arg (argp, unsigned *)  = value; goto gotvalue; }
-  if (!strcmp (mode, "%ld")) { *va_arg (argp, long *)      = value; goto gotvalue; }
-  if (!strcmp (mode, "%hd")) { *va_arg (argp, short *)     = value; goto gotvalue; }
-  if (!strcmp (mode, "%f"))  { *va_arg (argp, float *)     = value; goto gotvalue; }
-  if (!strcmp (mode, "%lf")) { *va_arg (argp, double *)    = value; goto gotvalue; }
+  if (!strcmp (mode, "%d"))  { *va_arg (argp, int *)       = value; return (TRUE); }
+  if (!strcmp (mode, "%u"))  { *va_arg (argp, unsigned *)  = value; return (TRUE); }
+  if (!strcmp (mode, "%ld")) { *va_arg (argp, long *)      = value; return (TRUE); }
+  if (!strcmp (mode, "%hd")) { *va_arg (argp, short *)     = value; return (TRUE); }
+  if (!strcmp (mode, "%f"))  { *va_arg (argp, float *)     = value; return (TRUE); }
+  if (!strcmp (mode, "%lf")) { *va_arg (argp, double *)    = value; return (TRUE); }
 
   /* no valid mode found */
   return (FALSE);
-
- gotvalue:
-  return (TRUE);
 }