[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [26048] trunk/blender/source/blender: have makesrna only replace newly generated files when their contents changes .

[Campbell Barton]

Revision: 26048
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=26048
Author:   campbellbarton
Date:     2010-01-17 15:04:40 +0100 (Sun, 17 Jan 2010)

Log Message:
-----------
have makesrna only replace newly generated files when their contents changes.
previously changing one rna_*.c file would rebuild them all making it slow to test small changes on the rna api.

also made errors in rna and dna generated give C defined #error's and line numbers that cause them.

Modified Paths:
--------------
    trunk/blender/source/blender/makesdna/intern/makesdna.c
    trunk/blender/source/blender/makesrna/intern/makesrna.c

Modified: trunk/blender/source/blender/makesdna/intern/makesdna.c
===================================================================
--- trunk/blender/source/blender/makesdna/intern/makesdna.c	2010-01-17 03:15:27 UTC (rev 26047)
+++ trunk/blender/source/blender/makesdna/intern/makesdna.c	2010-01-17 14:04:40 UTC (rev 26048)
@@ -1050,10 +1050,10 @@
 
 /* ************************* END MAKE DNA ********************** */
 
-static void make_bad_file(char *file)
+static void make_bad_file(char *file, int line)
 {
 	FILE *fp= fopen(file, "w");
-	fprintf(fp, "ERROR! Cannot make correct DNA.c file, STUPID!\n");
+	fprintf(fp, "#error \"Error! can't make correct DNA.c file from %s:%d, STUPID!\"\n", __FILE__, line);
 	fclose(fp);
 }
 
@@ -1087,7 +1087,7 @@
 			if (make_structDNA(baseDirectory, file)) {
 				// error
 				fclose(file);
-				make_bad_file(argv[1]);
+				make_bad_file(argv[1], __LINE__);
 				return_status = 1;
 			} else {
 				fprintf(file, "};\n");

Modified: trunk/blender/source/blender/makesrna/intern/makesrna.c
===================================================================
--- trunk/blender/source/blender/makesrna/intern/makesrna.c	2010-01-17 03:15:27 UTC (rev 26047)
+++ trunk/blender/source/blender/makesrna/intern/makesrna.c	2010-01-17 14:04:40 UTC (rev 26048)
@@ -44,6 +44,82 @@
 #endif
 #endif
 
+/* Replace if different */
+#define TMP_EXT ".tmp"
+
+static int replace_if_different(char *tmpfile)
+{
+
+#define REN_IF_DIFF \
+	if(rename(tmpfile, orgfile) != 0) { \
+		fprintf(stderr, "%s:%d, rename error: \"%s\" -> \"%s\"", __FILE__, __LINE__, tmpfile, orgfile); \
+		return -1; \
+	} \
+	remove(tmpfile); \
+	return 1; \
+
+
+	FILE *fp_new, *fp_org;
+	int len_new, len_org;
+	char *arr_new, *arr_org;
+	int cmp;
+
+	char orgfile[4096];
+
+	strcpy(orgfile, tmpfile);
+	orgfile[strlen(orgfile) - strlen(TMP_EXT)] = '\0'; /* strip '.tmp' */
+
+	fp_org= fopen(orgfile, "rb");
+
+	if(fp_org==NULL) {
+		REN_IF_DIFF;
+	}
+
+	fp_new= fopen(tmpfile, "rb");
+
+	if(fp_new==NULL) {
+		/* shouldn't happen, just to be safe */
+		fprintf(stderr, "%s:%d, open error: \"%s\"", __FILE__, __LINE__, tmpfile);
+		return -1;
+	}
+
+	fseek(fp_new, 0L, SEEK_END); len_new = ftell(fp_new); fseek(fp_new, 0L, SEEK_SET);
+	fseek(fp_org, 0L, SEEK_END); len_org = ftell(fp_org); fseek(fp_org, 0L, SEEK_SET);
+
+
+	if(len_new != len_org) {
+		fclose(fp_new);
+		fclose(fp_org);
+		REN_IF_DIFF;
+	}
+
+	/* now compare the files... */
+	arr_new= MEM_mallocN(sizeof(char)*len_new, "rna_cmp_file_new");
+	arr_org= MEM_mallocN(sizeof(char)*len_org, "rna_cmp_file_org");
+
+	fread(arr_new, sizeof(char), len_new, fp_new);
+	fread(arr_org, sizeof(char), len_org, fp_org);
+
+	fclose(fp_new);
+	fclose(fp_org);
+
+	cmp= memcmp(arr_new, arr_org, len_new);
+
+	MEM_freeN(arr_new);
+	MEM_freeN(arr_org);
+
+	if(cmp) {
+		REN_IF_DIFF;
+	}
+	else {
+		return 0;
+	}
+
+#undef REN_IF_DIFF
+}
+
+
+
 /* Sorting */
 
 static int cmp_struct(const void *a, const void *b)
@@ -2340,10 +2416,10 @@
 	fprintf(f, "}\n\n#endif /* __RNA_BLENDER_CPP_H__ */\n\n");
 }
 
-static void make_bad_file(char *file)
+static void make_bad_file(char *file, int line)
 {
 	FILE *fp= fopen(file, "w");
-	fprintf(fp, "ERROR! Cannot make correct RNA file, STUPID!\n");
+	fprintf(fp, "#error \"Error! can't make correct RNA file from %s:%d, STUPID!\"\n", __FILE__, line);
 	fclose(fp);
 }
 
@@ -2373,12 +2449,12 @@
 
 	/* create RNA_blender_cpp.h */
 	strcpy(deffile, outfile);
-	strcat(deffile, "RNA_blender_cpp.h");
+	strcat(deffile, "RNA_blender_cpp.h" TMP_EXT);
 
 	status= (DefRNA.error != 0);
 
 	if(status) {
-		make_bad_file(deffile);
+		make_bad_file(deffile, __LINE__);
 	}
 	else {
 		file = fopen(deffile, "w");
@@ -2390,11 +2466,12 @@
 		else {
 			rna_generate_header_cpp(brna, file);
 			fclose(file);
-
 			status= (DefRNA.error != 0);
 		}
 	}
 
+	replace_if_different(deffile);
+
 	rna_sort(brna);
 
 	/* create rna_gen_*.c files */
@@ -2402,10 +2479,10 @@
 		strcpy(deffile, outfile);
 		strcat(deffile, PROCESS_ITEMS[i].filename);
 		deffile[strlen(deffile)-2] = '\0';
-		strcat(deffile, "_gen.c");
+		strcat(deffile, "_gen.c" TMP_EXT);
 
 		if(status) {
-			make_bad_file(deffile);
+			make_bad_file(deffile, __LINE__);
 		}
 		else {
 			file = fopen(deffile, "w");
@@ -2417,18 +2494,19 @@
 			else {
 				rna_generate(brna, file, PROCESS_ITEMS[i].filename, PROCESS_ITEMS[i].api_filename);
 				fclose(file);
-
 				status= (DefRNA.error != 0);
 			}
 		}
+
+		replace_if_different(deffile);
 	}
 
 	/* create RNA_blender.h */
 	strcpy(deffile, outfile);
-	strcat(deffile, "RNA_blender.h");
+	strcat(deffile, "RNA_blender.h" TMP_EXT);
 
 	if(status) {
-		make_bad_file(deffile);
+		make_bad_file(deffile, __LINE__);
 	}
 	else {
 		file = fopen(deffile, "w");
@@ -2440,11 +2518,12 @@
 		else {
 			rna_generate_header(brna, file);
 			fclose(file);
-
 			status= (DefRNA.error != 0);
 		}
 	}
 
+	replace_if_different(deffile);
+
 	/* free RNA */
 	RNA_define_free(brna);
 	RNA_free(brna);