Changeset 32913bc for libcfa

Timestamp:

Aug 31, 2021, 11:54:07 AM (3 years ago)

Author:

Jacob Prud'homme <jafprudhomme@…>

Branches:

ADT, ast-experimental, enum, forall-pointer-decay, jacob/cs343-translation, master, pthread-emulation, qualifiedEnum

Children:

e22c841

Parents:

692db791

Message:

Fixed up comments

File:

• libcfa/src/parseconfig.cfa (modified) (3 diffs)

libcfa/src/parseconfig.cfa

@@ -9 +9 @@
 
 
+// TODO: Add names of missing config entries to exception (see further below)
 static vtable(Missing_Config_Entries) Missing_Config_Entries_vt;
 
@@ -113 +114 @@
 } // comments
 
-// Parse configuration from a file formatted in shell style
+// Parse configuration from a file formatted in tabular (CS 343) style
 KVPairs * parse_tabular_config_format( const char * config_file, size_t num_entries ) {
         // TODO: Change this to a unique_ptr when we fully support returning them (move semantics)
@@ -126 +127 @@
 
                 while () {                                                                      // parameter names can appear in any order
-                        // Must add check to see if already read in a key-value pair,
+                        // NOTE: Must add check to see if already read in value for this key,
                         // once we switch to using hash table as intermediate storage
                   if ( comments( in, key ) ) break;                     // eof ?