diff options
author | Andy Getzendanner <durandal@google.com> | 2022-11-16 14:47:18 -0800 |
---|---|---|
committer | Copybara-Service <copybara-worker@google.com> | 2022-11-16 14:48:00 -0800 |
commit | 0c048353d2e7347f148228d785c3f0a8c81e2223 (patch) | |
tree | 2c6c5a87eccd672de83b8c631c8dcd834c54883e /absl/log/internal/log_message.cc | |
parent | ae52431653d1d2815820680d4539e749f53b7bc7 (diff) | |
download | abseil-0c048353d2e7347f148228d785c3f0a8c81e2223.tar.gz abseil-0c048353d2e7347f148228d785c3f0a8c81e2223.tar.bz2 abseil-0c048353d2e7347f148228d785c3f0a8c81e2223.zip |
Minor formatting changes in preparation for structured logging...
PiperOrigin-RevId: 489044912
Change-Id: I1657dd60bcfa2d0fb4b889f27f4f237325b73f08
Diffstat (limited to 'absl/log/internal/log_message.cc')
-rw-r--r-- | absl/log/internal/log_message.cc | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/absl/log/internal/log_message.cc b/absl/log/internal/log_message.cc index 648086e9..8329597e 100644 --- a/absl/log/internal/log_message.cc +++ b/absl/log/internal/log_message.cc @@ -42,7 +42,6 @@ #include "absl/debugging/internal/examine_stack.h" #include "absl/log/globals.h" #include "absl/log/internal/append_truncated.h" -#include "absl/log/internal/config.h" #include "absl/log/internal/globals.h" #include "absl/log/internal/log_format.h" #include "absl/log/internal/log_sink_set.h" @@ -87,8 +86,8 @@ void WriteToStream(const char* data, void* os) { // A write-only `std::streambuf` that writes into an `absl::Span<char>`. // -// This class is responsible for writing a metadata prefix just before the first -// data are streamed in. The metadata are subject to change (cf. +// This class is responsible for writing a metadata prefix just before the +// first data are streamed in. The metadata are subject to change (cf. // `LogMessage::AtLocation`) until then, so we wait as long as possible. // // This class is also responsible for reserving space for a trailing newline @@ -96,7 +95,8 @@ void WriteToStream(const char* data, void* os) { // streamed in. class LogEntryStreambuf final : public std::streambuf { public: - explicit LogEntryStreambuf(absl::Span<char> buf, const absl::LogEntry& entry) + explicit LogEntryStreambuf(absl::Span<char> buf, + const absl::LogEntry& entry) : buf_(buf), entry_(entry), prefix_len_(0), finalized_(false) { // To detect when data are first written, we leave the put area null, // override `overflow`, and check ourselves in `xsputn`. @@ -107,7 +107,8 @@ class LogEntryStreambuf final : public std::streambuf { absl::Span<const char> Finalize() { assert(!finalized_); - // If no data were ever streamed in, this is where we must write the prefix. + // If no data were ever streamed in, this is where we must write the + // prefix. if (pbase() == nullptr) Initialize(); // Here we reclaim the two bytes we reserved. ptrdiff_t idx = pptr() - pbase(); @@ -140,8 +141,8 @@ class LogEntryStreambuf final : public std::streambuf { private: void Initialize() { - // Here we reserve two bytes in our buffer to guarantee `Finalize` space to - // add a trailing "\n\0". + // Here we reserve two bytes in our buffer to guarantee `Finalize` space + // to add a trailing "\n\0". assert(buf_.size() >= 2); setp(buf_.data(), buf_.data() + buf_.size() - 2); if (entry_.prefix()) { @@ -157,7 +158,8 @@ class LogEntryStreambuf final : public std::streambuf { } size_t Append(absl::string_view data) { - absl::Span<char> remaining(pptr(), static_cast<size_t>(epptr() - pptr())); + absl::Span<char> remaining(pptr(), + static_cast<size_t>(epptr() - pptr())); const size_t written = log_internal::AppendTruncated(data, remaining); pbump(static_cast<int>(written)); return written; @@ -202,7 +204,7 @@ LogMessage::LogMessageData::LogMessageData(const char* file, int line, absl::LogSeverity severity, absl::Time timestamp) : extra_sinks_only(false), - streambuf_(absl::MakeSpan(string_buf), entry) { + streambuf_(absl::MakeSpan(string_buf), entry) { entry.full_filename_ = file; entry.base_filename_ = Basename(file); entry.line_ = line; @@ -216,8 +218,7 @@ LogMessage::LogMessageData::LogMessageData(const char* file, int line, LogMessage::LogMessage(const char* file, int line, absl::LogSeverity severity) : data_( absl::make_unique<LogMessageData>(file, line, severity, absl::Now())) - , - stream_(&data_->streambuf_) +, stream_(&data_->streambuf_) { data_->first_fatal = false; data_->is_perror = false; @@ -225,8 +226,8 @@ LogMessage::LogMessage(const char* file, int line, absl::LogSeverity severity) // Legacy defaults for LOG's ostream: stream_.setf(std::ios_base::showbase | std::ios_base::boolalpha); - // `fill('0')` is omitted here because its effects are very different without - // structured logging. Resolution is tracked in b/111310488. + // `fill('0')` is omitted here because its effects are very different + // without structured logging. Resolution is tracked in b/111310488. // This logs a backtrace even if the location is subsequently changed using // AtLocation. This quirk, and the behavior when AtLocation is called twice, |