Changeset 02a22a2 for libcfa

Timestamp:

Aug 31, 2021, 1:49:09 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:

ba61cd9

Parents:

98d6b744

git-author:

Jacob Prud'homme <jafprudhomme@…> (07/27/21 12:38:33)

git-committer:

Jacob Prud'homme <jafprudhomme@…> (08/31/21 01:49:09)

Message:

Reverted to using reference to KVPairs

This will have to be changed in the future as we are returning a
stack-allocated value out of scope

File:

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

libcfa/src/parseconfig.cfa

@@ -45 +45 @@
 }
 
-void add_kv_pair( KVPairs kv_pairs, char * k, char * v ) with ( kv_pairs ) {
+void add_kv_pair( KVPairs & kv_pairs, char * k, char * v ) with ( kv_pairs ) {
         if ( size == max_size ) {
                 max_size *= 2;
@@ -67 +67 @@
 // Parse configuration from a file formatted in shell style
 KVPairs & parse_shell_config_format( const char * config_file, size_t num_entries ) {
-        * KVPairs kv_pairs;// = { num_entries };
+        // * KVPairs kv_pairs;
+        KVPairs kv_pairs = { num_entries };
 
         ifstream in;
@@ -80 +81 @@
                         in | value;
 
-                        add_kv_pair( *kv_pairs, key, value );
+                        add_kv_pair( kv_pairs, key, value );
 
                   if ( fail( in ) ) break;
@@ -91 +92 @@
         close( in );
 
-        return *kv_pairs;
+        return kv_pairs;
 }