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;                        \