From 7454bdded7878b10dcdf8d5c64756c7a87e46500 Mon Sep 17 00:00:00 2001 From: Abseil Team Date: Thu, 10 May 2018 10:16:01 -0700 Subject: - be60293bacf8d17861853a73c751b4a74534b049 Update header file for failure_signal_handler.h by Tom Manshreck - 83dbeb97cb3874932a4159f2f287c5b2cca7089b Fix for MSVC warning C4244 about "conversion from 'int' t... by Abseil Team GitOrigin-RevId: be60293bacf8d17861853a73c751b4a74534b049 Change-Id: If8a91fa3edca47f349662cb214fde380c65802bb --- absl/debugging/failure_signal_handler.h | 110 ++++++++++++++++++-------------- absl/strings/str_cat.h | 2 +- 2 files changed, 63 insertions(+), 49 deletions(-) diff --git a/absl/debugging/failure_signal_handler.h b/absl/debugging/failure_signal_handler.h index 17522f00..c57954e5 100644 --- a/absl/debugging/failure_signal_handler.h +++ b/absl/debugging/failure_signal_handler.h @@ -1,4 +1,3 @@ -// // Copyright 2018 The Abseil Authors. // // Licensed under the Apache License, Version 2.0 (the "License"); @@ -13,85 +12,100 @@ // See the License for the specific language governing permissions and // limitations under the License. // - -// This module allows the programmer to install a signal handler that -// dumps useful debugging information (like a stacktrace) on program -// failure. To use this functionality, call -// absl::InstallFailureSignalHandler() very early in your program, -// usually in the first few lines of main(): +// ----------------------------------------------------------------------------- +// File: failure_signal_handler.h +// ----------------------------------------------------------------------------- +// +// This file configures the Abseil *failure signal handler* to capture and dump +// useful debugging information (such as a stacktrace) upon program failure. +// +// To use the failure signal handler, call `absl::InstallFailureSignalHandler()` +// very early in your program, usually in the first few lines of main(): // // int main(int argc, char** argv) { +// // Initialize the symbolizer to get a human-readable stack trace // absl::InitializeSymbolizer(argv[0]); +// // absl::FailureSignalHandlerOptions options; // absl::InstallFailureSignalHandler(options); // DoSomethingInteresting(); // return 0; // } +// +// Any program that raises a fatal signal (such as `SIGSEGV`, `SIGILL`, +// `SIGFPE`, `SIGABRT`, `SIGTERM`, `SIGBUG`, and `SIGTRAP`) will call the +// installed failure signal handler and provide debugging information to stderr. +// +// Note that you should *not* install the Abseil failure signal handler more +// than once. You may, of course, have another (non-Abseil) failure signal +// handler installed (which would be triggered if Abseil's failure signal +// handler sets `call_previous_handler` to `true`). #ifndef ABSL_DEBUGGING_FAILURE_SIGNAL_HANDLER_H_ #define ABSL_DEBUGGING_FAILURE_SIGNAL_HANDLER_H_ namespace absl { -// Options struct for absl::InstallFailureSignalHandler(). +// FailureSignalHandlerOptions +// +// Struct for holding `absl::InstallFailureSignalHandler()` configuration +// options. struct FailureSignalHandlerOptions { - // If true, try to symbolize the stacktrace emitted on failure. + // If true, try to symbolize the stacktrace emitted on failure, provided that + // you have initialized a symbolizer for that purpose. (See symbolize.h for + // more information.) bool symbolize_stacktrace = true; - // If true, try to run signal handlers on an alternate stack (if - // supported on the given platform). This is useful in the case - // where the program crashes due to a stack overflow. By running on - // a alternate stack, the signal handler might be able to run even - // when the normal stack space has been exausted. The downside of - // using an alternate stack is that extra memory for the alternate - // stack needs to be pre-allocated. + // If true, try to run signal handlers on an alternate stack (if supported on + // the given platform). An alternate stack is useful for program crashes due + // to a stack overflow; by running on a alternate stack, the signal handler + // may run even when normal stack space has been exausted. The downside of + // using an alternate stack is that extra memory for the alternate stack needs + // to be pre-allocated. bool use_alternate_stack = true; - // If positive, FailureSignalHandler() sets an alarm to be delivered - // to the program after this many seconds, which will immediately - // abort the program. This is useful in the potential case where - // FailureSignalHandler() itself is hung or deadlocked. + // If positive, indicates the number of seconds after which the failure signal + // handler is invoked to abort the program. Setting such an alarm is useful in + // cases where the failure signal handler itself may become hung or + // deadlocked. int alarm_on_failure_secs = 3; - // If false, after absl::FailureSignalHandler() runs, the signal is - // raised to the default handler for that signal (which normally - // terminates the program). + // If true, call the previously registered signal handler for the signal that + // was received (if one was registered) after the existing signal handler + // runs. This mechanism can be used to chain signal handlers together. // - // If true, after absl::FailureSignalHandler() runs, it will call - // the previously registered signal handler for the signal that was - // received (if one was registered). This can be used to chain - // signal handlers. + // If false, the signal is raised to the default handler for that signal + // (which normally terminates the program). // - // IMPORTANT: If true, the chained fatal signal handlers must not - // try to recover from the fatal signal. Instead, they should - // terminate the program via some mechanism, like raising the - // default handler for the signal, or by calling _exit(). - // absl::FailureSignalHandler() may put parts of the Abseil - // library into a state that cannot be recovered from. + // IMPORTANT: If true, the chained fatal signal handlers must not try to + // recover from the fatal signal. Instead, they should terminate the program + // via some mechanism, like raising the default handler for the signal, or by + // calling `_exit()`. Note that the failure signal handler may put parts of + // the Abseil library into a state from which they cannot recover. bool call_previous_handler = false; - // If not null, this function may be called with a std::string argument - // containing failure data. This function is used as a hook to write - // the failure data to a secondary location, for instance, to a log - // file. This function may also be called with a null data - // argument. This is a hint that this is a good time to flush any - // buffered data before the program may be terminated. Consider + // If non-null, indicates a pointer to a callback function that will be called + // upon failure, with a std::string argument containing failure data. This function + // may be used as a hook to write failure data to a secondary location, such + // as a log file. This function may also be called with null data, as a hint + // to flush any buffered data before the program may be terminated. Consider // flushing any buffered data in all calls to this function. // - // Since this function runs in a signal handler, it should be + // Since this function runs within a signal handler, it should be // async-signal-safe if possible. // See http://man7.org/linux/man-pages/man7/signal-safety.7.html void (*writerfn)(const char*) = nullptr; }; -// Installs a signal handler for the common failure signals SIGSEGV, -// SIGILL, SIGFPE, SIGABRT, SIGTERM, SIGBUG, and SIGTRAP (if they -// exist on the given platform). The signal handler dumps program -// failure data in a unspecified format to stderr. The data dumped by -// the signal handler includes information that may be useful in -// debugging the failure. This may include the program counter, a -// stacktrace, and register information on some systems. Do not rely -// on the exact format of the output; it is subject to change. +// InstallFailureSignalHandler() +// +// Installs a signal handler for the common failure signals `SIGSEGV`, `SIGILL`, +// `SIGFPE`, `SIGABRT`, `SIGTERM`, `SIGBUG`, and `SIGTRAP` (provided they exist +// on the given platform). The failure signal handler dumps program failure data +// useful for debugging in an unspecified format to stderr. This data may +// include the program counter, a stacktrace, and register information on some +// systems; do not rely on an exact format for the output, as it is subject to +// change. void InstallFailureSignalHandler(const FailureSignalHandlerOptions& options); namespace debugging_internal { diff --git a/absl/strings/str_cat.h b/absl/strings/str_cat.h index e38369ca..e5501a50 100644 --- a/absl/strings/str_cat.h +++ b/absl/strings/str_cat.h @@ -80,7 +80,7 @@ struct AlphaNumBuffer { // `Dec` conversion and fill character to use. A `kZeroPad2` value, for example, // would produce hexadecimal strings such as "0A","0F" and a 'kSpacePad5' value // would produce hexadecimal strings such as " A"," F". -enum PadSpec { +enum PadSpec : uint8_t { kNoPad = 1, kZeroPad2, kZeroPad3, -- cgit v1.2.3