Changeset 40409 for trunk

Timestamp:

Apr 26, 2018, 12:37:16 PM (8 years ago)

Author:

eugene

Message:

add some comments

File:

• trunk/Ohana/src/tools/src/fields.c (modified) (2 diffs)

trunk/Ohana/src/tools/src/fields.c

@@ -96 +96 @@
           }
         }
+        // we are checking each extension to see if it matches the regex.
         if (!regexec (&preg, extname, 0, NULL, 0)) {
           GotField &= print_fields (filename, extname, &header, argc, argv);
@@ -105 +106 @@
         Nextend ++;
 
+        // this implementation is very inefficient!
+        // since we are checking each header for a regex, we should
+        // loop over all extensions once.  the function below reads the
+        // entire file a second time
         GotFile &= gfits_read_Xheader (filename, &header, Nextend);
         continue;