From e9ff3f2e6545bbce636210ad2ae30d8e3eb05124 Mon Sep 17 00:00:00 2001 From: ridiculousfish Date: Thu, 22 Jul 2021 09:44:50 -0700 Subject: [PATCH] Remove a stale comment. --- src/common.h | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/common.h b/src/common.h index 0cf7ce86e..f006762eb 100644 --- a/src/common.h +++ b/src/common.h @@ -198,10 +198,6 @@ extern const bool has_working_tty_timestamps; extern const wcstring g_empty_string; // Pause for input, then exit the program. If supported, print a backtrace first. -// The `return` will never be run but silences oclint warnings. Especially when this is called -// from within a `switch` block. As of the time I'm writing this oclint doesn't recognize the -// `__attribute__((noreturn))` on the exit_without_destructors() function. -// TODO: we use C++11 [[noreturn]] now, does that change things? #define FATAL_EXIT() \ do { \ char exit_read_buff; \