Merge pull request #3664 from youngar/master

Reduce default output of tracegen
eclipse-omr · Mar 20, 2019 · d4ac770 · d4ac770
2 parents 62adf25 + 64fcd7a
commit d4ac770
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/tools/tracegen/CFileWriter.cpp b/tools/tracegen/CFileWriter.cpp
@@ -45,7 +45,9 @@ CFileWriter::writeOutputFiles(J9TDFOptions *options, J9TDFFile *tdf, J9TDFGroup
 	time_t targetFileMtime = FileUtils::getMtime(fileName);
 
 	if ((false == options->force) && (targetFileMtime > sourceFileMtime)) {
-		printf("C file is already up-to-date: %s\n", fileName);
+		if (options->verboseOutput) {
+			printf("C file is already up-to-date: %s\n", fileName);
+		}
 		Port::omrmem_free((void **)&fileName);
 		return RC_OK;
 	}

diff --git a/tools/tracegen/DATFileWriter.cpp b/tools/tracegen/DATFileWriter.cpp
@@ -49,7 +49,9 @@ DATFileWriter::writeOutputFiles(J9TDFOptions *options, J9TDFFile *tdf)
 	time_t targetFileMtime = FileUtils::getMtime(fileName);
 
 	if ((false == options->force) && (targetFileMtime > sourceFileMtime)) {
-		printf("pdat file is already up-to-date: %s\n", fileName);
+		if (options->verboseOutput) {
+			printf("pdat file is already up-to-date: %s\n", fileName);
+		}
 		Port::omrmem_free((void **)&fileName);
 		return RC_OK;
 	}

diff --git a/tools/tracegen/TraceHeaderWriter.cpp b/tools/tracegen/TraceHeaderWriter.cpp
@@ -141,12 +141,14 @@ TraceHeaderWriter::writeOutputFiles(J9TDFOptions *options, J9TDFFile *tdf)
 	time_t targetFileMtime = FileUtils::getMtime(fileName);
 
 	if (false == options->force && targetFileMtime > sourceFileMtime) {
-		printf("Header file is already up-to-date: %s\n", fileName);
+		if (options->verboseOutput) {
+			printf("Header file is already up-to-date: %s\n", fileName);
+		}
 		Port::omrmem_free((void **)&fileName);
 		return RC_OK;
 	}
 
-	if (true == options->verboseOutput) {
+	if (options->verboseOutput) {
 		printf("Creating header file: %s\n", fileName);
 	}