devtools/create-gossipstore: clean up enough to pass check-source.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

devtools/create-gossipstore.c

@@ -9,8 +9,6 @@
 #include <gossipd/gen_gossip_store.h>
 #include <gossipd/gossip_store.h>
 #include <inttypes.h>
-#include <stdio.h>
-#include <stdlib.h>
 #include <sys/stat.h>
 #include <sys/types.h>
 #include <unistd.h>
@@ -26,7 +24,7 @@ struct scidsat * load_scid_file(FILE * scidfd)
         fscanf(scidfd, "%s\n", title);	
 	struct scidsat * scids = calloc(n, sizeof(scidsat));
 	int i = 0;
-        while(fscanf(scidfd, "%s ,%ld\n", scids[i].scid, &scids[i].sat.satoshis) == 2 ) {
+        while(fscanf(scidfd, "%s ,%ld\n", scids[i].scid, &scids[i].sat.satoshis) == 2 ) { /* Raw: read from file */
 			i++;
 	}
 	return scids;
@@ -41,7 +39,7 @@ int main(int argc, char *argv[])
 	char *infile = NULL, *outfile = NULL, *scidfile = NULL, *csat = NULL;
 	int infd, outfd;
        	FILE * scidfd;
-	struct scidsat * scids;
+	struct scidsat * scids = NULL;
 	unsigned max = -1U;
 
 	setup_locale();
@@ -113,7 +111,7 @@ int main(int argc, char *argv[])
 
 		switch (fromwire_peektype(inmsg)) {
 		case WIRE_CHANNEL_ANNOUNCEMENT:
-			if (scidfile) {
+			if (scids) {
 				sat = scids[scidi].sat;
 				scidi += 1;
 			}
@@ -151,7 +149,6 @@ int main(int argc, char *argv[])
 			break;
 	}
 	fprintf(stderr, "channels %d, updates %d, nodes %d\n", channels, updates, nodes);
-	if (scidfile)
+	free(scids);
-                free(scids);
 	return 0;
 }

devtools/create-gossipstore.h

@@ -1,3 +1,5 @@
+#ifndef LIGHTNING_DEVTOOLS_CREATE_GOSSIPSTORE_H
+#define LIGHTNING_DEVTOOLS_CREATE_GOSSIPSTORE_H
 #include <stdlib.h>
 #include <stdio.h>
 #include <common/amount.h>
@@ -8,4 +10,4 @@ struct scidsat {
 } scidsat;
 
 struct scidsat * load_scid_file(FILE * scidfd);
-
+#endif /* LIGHTNING_DEVTOOLS_CREATE_GOSSIPSTORE_H */

tools/check-includes.sh

@@ -9,7 +9,7 @@ HEADER_ID_SUFFIX="_H"
 REGEXP_EXCLUDE_FILES_WITH_PREFIX="ccan/"
 for HEADER_FILE in $(git ls-files -- "*.h" | grep -vE "^${REGEXP_EXCLUDE_FILES_WITH_PREFIX}")
 do
-    HEADER_ID_BASE=$(tr / _ <<< "${HEADER_FILE/%.h/}" | tr "[:lower:]" "[:upper:]")
+    HEADER_ID_BASE=$(tr /- _ <<< "${HEADER_FILE/%.h/}" | tr "[:lower:]" "[:upper:]")
     HEADER_ID="${HEADER_ID_PREFIX}${HEADER_ID_BASE}${HEADER_ID_SUFFIX}"
     if [[ $(grep -cE "^#((ifndef|define) ${HEADER_ID}|endif /\\* ${HEADER_ID} \\*/)$" "${HEADER_FILE}") != 3 ]]; then
         echo "${HEADER_FILE} seems to be missing the expected include guard:"