Commit 14240d16 authored by Mason Freed's avatar Mason Freed Committed by Commit Bot

Add helpful comment about enable-logging=stderr

I have run into this twice myself, and I always end up reading
the comments next to VLOG() to remind myself what I'm missing.
They don't clue you in to the fact that VLOG() statements require
enable-logging=stderr or they won't be seen.

Change-Id: I8d00da46bf1555e0d6f5728cdc258f1ce36e3d64
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2346927
Commit-Queue: kylechar <kylechar@chromium.org>
Reviewed-by: default avatarkylechar <kylechar@chromium.org>
Reviewed-by: default avatarPhilip Rogers <pdr@chromium.org>
Auto-Submit: Mason Freed <masonfreed@chromium.org>
Cr-Commit-Position: refs/heads/master@{#797213}
parent fb889f37
...@@ -105,6 +105,10 @@ ...@@ -105,6 +105,10 @@
// E.g., "*/foo/bar/*=2" would change the logging level for all code // E.g., "*/foo/bar/*=2" would change the logging level for all code
// in source files under a "foo/bar" directory. // in source files under a "foo/bar" directory.
// //
// Note that for a Chromium binary built in release mode (is_debug = false) you
// must pass "--enable-logging=stderr" in order to see the output of VLOG
// statements.
//
// There's also VLOG_IS_ON(n) "verbose level" condition macro. To be used as // There's also VLOG_IS_ON(n) "verbose level" condition macro. To be used as
// //
// if (VLOG_IS_ON(2)) { // if (VLOG_IS_ON(2)) {
...@@ -157,6 +161,9 @@ ...@@ -157,6 +161,9 @@
// //
// Note that the visibility can be changed by setting preferences in // Note that the visibility can be changed by setting preferences in
// SetLogItems() // SetLogItems()
//
// Additional logging-related information can be found here:
// https://chromium.googlesource.com/chromium/src/+/master/docs/linux/debugging.md#Logging
namespace logging { namespace logging {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment