makesdna/makesrna: silence output by default
authorCampbell Barton <ideasman42@gmail.com>
Tue, 28 Nov 2017 09:27:40 +0000 (20:27 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 28 Nov 2017 09:27:40 +0000 (20:27 +1100)
No need to print status for basic & reliable operations,
build systems can output operations they run if needed,
or debug output changed in the source if developers are debugging.

Nice for ninja, so any printed text hints at a problem to fix.

source/blender/makesdna/intern/makesdna.c
source/blender/makesrna/intern/makesrna.c

index 2cea8715a657ca6667e26866d79b96c91fe75972..17e30de4509b6cd9a3838dea8e176b1695fd3eeb 100644 (file)
@@ -978,7 +978,7 @@ static int make_structDNA(const char *baseDirectory, FILE *file, FILE *file_offs
        char str[SDNA_MAX_FILENAME_LENGTH], *cp;
        int firststruct;
        
-       if (debugSDNA > -1) {
+       if (debugSDNA > 0) {
                fflush(stdout);
                printf("Running makesdna at debug level %d\n", debugSDNA);
        }
@@ -1074,7 +1074,7 @@ static int make_structDNA(const char *baseDirectory, FILE *file, FILE *file_offs
 
        /* file writing */
 
-       if (debugSDNA > -1) printf("Writing file ... ");
+       if (debugSDNA > 0) printf("Writing file ... ");
                
        if (nr_names == 0 || nr_structs == 0) {
                /* pass */
@@ -1184,7 +1184,7 @@ static int make_structDNA(const char *baseDirectory, FILE *file, FILE *file_offs
        MEM_freeN(typelens_64);
        MEM_freeN(structs);
 
-       if (debugSDNA > -1) printf("done.\n");
+       if (debugSDNA > 0) printf("done.\n");
        
        return(0);
 }
index 2c558e56412a64b808010db852f7c5a06371da88..ff081a56b61afb8a65fd291ae41f8dab39983b01 100644 (file)
 #  endif
 #endif
 
+/**
+ * Variable to control debug output of makesrna.
+ * debugSRNA:
+ *  - 0 = no output, except errors
+ *  - 1 = detail actions
+ */
+static int debugSRNA = 0;
+
 /* stub for BLI_abort() */
 #ifndef NDEBUG
 void BLI_system_backtrace(FILE *fp)
@@ -62,7 +70,9 @@ void BLI_system_backtrace(FILE *fp)
 static int file_older(const char *file1, const char *file2)
 {
        struct stat st1, st2;
-       /* printf("compare: %s %s\n", file1, file2); */
+       if (debugSRNA > 0) {
+               printf("compare: %s %s\n", file1, file2);
+       }
 
        if (stat(file1, &st1)) return 0;
        if (stat(file2, &st2)) return 0;
@@ -4134,7 +4144,9 @@ int main(int argc, char **argv)
                return_status = 1;
        }
        else {
-               fprintf(stderr, "Running makesrna\n");
+               if (debugSRNA > 0) {
+                       fprintf(stderr, "Running makesrna\n");
+               }
                makesrna_path = argv[0];
                return_status = rna_preprocess(argv[1]);
        }