fix review

Signed-off-by: Matthias Gatto <[email protected]>

cognac_gen.sh

@@ -328,10 +328,11 @@ EOF
 	}
 EOF
 		    else
-			# no type check are made here, the aditional stuff is assumed to be a string
+			# no type checks are made here, the additional stuff is assumed to be a string
 			cat <<EOF
 	{
 		struct additional_strings *elem = malloc(sizeof *elem);
+		TRY(!elem, "Memory allocation failed");
 		(void)aret;
 		ptr_array_append(pa, elem);
 

construct_data.c.sh

@@ -18,7 +18,7 @@ fi
 aditional=$(json-search -n additionalProperties <<< $base)
 
 if [ "$aditional" != "null"  -a  "$aditional" != "false" ]; then
-    # no type check are made here, the aditional stuff is assumed to be a string
+    # no type checks are made here, the additional stuff is assumed to be a string
     cat <<EOF
 	struct additional_strings **elems = args->additional_strs;
 

main-helper.h

@@ -42,9 +42,8 @@ struct ptr_array {
  */
 #define OBJ_SIZE 8
 
-/* a is an array of "flexible size", v is the value to push, and pa
- * is use to store the pointer somewhere so we can free everything
- * esilly
+/* a is an array with a flexible size, v is the value to push,
+ * and pa is used to store the pointer so that we can easily free everything later.
  */
 #define SET_NEXT(a,v,pa) do {						\
 		int cnt;						\

mk_args.c.sh

@@ -78,7 +78,7 @@ write_struct() {
     aditional=$(json-search -n additionalProperties <<< $st_info)
 
     if [ "$aditional" != "null"  -a  "$aditional" != "false" ]; then
-	# no type check are made here, the aditional stuff is assumed to be a string
+	# no type checks are made here, the additional stuff is assumed to be a string
 	echo -e '\tstruct additional_strings **additional_strs;'
     fi