[Bf-blender-cvs] [0511de99bb1] master: Fix non-working verbosity when set prior to --debug
Sergey Sharybin
noreply at git.blender.org
Fri Jun 28 15:45:33 CEST 2019
Commit: 0511de99bb1174ded5c08825ff29881daa7317e2
Author: Sergey Sharybin
Date: Fri Jun 28 15:42:58 2019 +0200
Branches: master
https://developer.blender.org/rB0511de99bb1174ded5c08825ff29881daa7317e2
Fix non-working verbosity when set prior to --debug
Before this change doing something like `--verbose 10 --debug-cycles`
did not properly set verbosity, only using those arguments in an other
way around was leading to a correct verbosity level.
===================================================================
M intern/cycles/util/util_logging.cpp
M intern/libmv/intern/logging.cc
===================================================================
diff --git a/intern/cycles/util/util_logging.cpp b/intern/cycles/util/util_logging.cpp
index 4a5e7e6a9ea..783a372e59f 100644
--- a/intern/cycles/util/util_logging.cpp
+++ b/intern/cycles/util/util_logging.cpp
@@ -17,6 +17,7 @@
#include "util/util_logging.h"
#include "util/util_math.h"
+#include "util/util_string.h"
#include <stdio.h>
#ifdef _MSC_VER
@@ -25,6 +26,17 @@
CCL_NAMESPACE_BEGIN
+static bool is_verbosity_set()
+{
+ using CYCLES_GFLAGS_NAMESPACE::GetCommandLineOption;
+
+ std::string verbosity;
+ if (!GetCommandLineOption("v", &verbosity)) {
+ return false;
+ }
+ return verbosity != "0";
+}
+
void util_logging_init(const char *argv0)
{
#ifdef WITH_CYCLES_LOGGING
@@ -36,7 +48,9 @@ void util_logging_init(const char *argv0)
google::InitGoogleLogging(argv0);
SetCommandLineOption("logtostderr", "1");
- SetCommandLineOption("v", "0");
+ if (!is_verbosity_set()) {
+ SetCommandLineOption("v", "0");
+ }
SetCommandLineOption("stderrthreshold", severity_fatal);
SetCommandLineOption("minloglevel", severity_fatal);
#else
@@ -49,7 +63,9 @@ void util_logging_start()
#ifdef WITH_CYCLES_LOGGING
using CYCLES_GFLAGS_NAMESPACE::SetCommandLineOption;
SetCommandLineOption("logtostderr", "1");
- SetCommandLineOption("v", "2");
+ if (!is_verbosity_set()) {
+ SetCommandLineOption("v", "2");
+ }
SetCommandLineOption("stderrthreshold", "1");
SetCommandLineOption("minloglevel", "0");
#endif
diff --git a/intern/libmv/intern/logging.cc b/intern/libmv/intern/logging.cc
index 6be0ef44e42..350be29d4db 100644
--- a/intern/libmv/intern/logging.cc
+++ b/intern/libmv/intern/logging.cc
@@ -23,6 +23,16 @@
#include "intern/utildefines.h"
#include "libmv/logging/logging.h"
+static bool is_verbosity_set() {
+ using LIBMV_GFLAGS_NAMESPACE::GetCommandLineOption;
+
+ std::string verbosity;
+ if (!GetCommandLineOption("v", &verbosity)) {
+ return false;
+ }
+ return verbosity != "0";
+}
+
void libmv_initLogging(const char* argv0) {
using LIBMV_GFLAGS_NAMESPACE::SetCommandLineOption;
// Make it so ERROR messages are always print into console.
@@ -30,8 +40,9 @@ void libmv_initLogging(const char* argv0) {
snprintf(severity_fatal, sizeof(severity_fatal), "%d",
google::GLOG_ERROR);
google::InitGoogleLogging(argv0);
- SetCommandLineOption("logtostderr", "1");
- SetCommandLineOption("v", "0");
+ if (!is_verbosity_set()) {
+ SetCommandLineOption("v", "0");
+ }
SetCommandLineOption("stderrthreshold", severity_fatal);
SetCommandLineOption("minloglevel", severity_fatal);
}
@@ -39,7 +50,9 @@ void libmv_initLogging(const char* argv0) {
void libmv_startDebugLogging(void) {
using LIBMV_GFLAGS_NAMESPACE::SetCommandLineOption;
SetCommandLineOption("logtostderr", "1");
- SetCommandLineOption("v", "2");
+ if (!is_verbosity_set()) {
+ SetCommandLineOption("v", "2");
+ }
SetCommandLineOption("stderrthreshold", "1");
SetCommandLineOption("minloglevel", "0");
}
More information about the Bf-blender-cvs
mailing list