]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
dout: Remove VLA sanity check
authorAdam C. Emerson <aemerson@redhat.com>
Fri, 22 Dec 2017 23:44:57 +0000 (18:44 -0500)
committerAdam C. Emerson <aemerson@redhat.com>
Mon, 8 Jan 2018 20:30:05 +0000 (15:30 -0500)
This doesn't seem to be able to work for C++17 under both GCC and
Clang. Maybe there's a better way to do it. I might try routing
that argument through a constexpr function in the macro later.

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
src/common/dout.h

index 60bee71122ae5c376d56bdfe663ea767eed7dfda..56e768d9afe7cc228a9d6c7d821f63fccc559abf 100644 (file)
@@ -49,9 +49,6 @@ public:
 #define dout_impl(cct, sub, v)                                         \
   do {                                                                 \
   if (cct->_conf->subsys.should_gather(sub, v)) {                      \
-    if (0) {                                                           \
-      [[maybe_unused]] char __array[((v >= -1) && (v <= 200)) ? 0 : -1] = {}; \
-    }                                                                  \
     static size_t _log_exp_length = 80;                                \
     ceph::logging::Entry *_dout_e = cct->_log->create_entry(v, sub, &_log_exp_length); \
     static_assert(std::is_convertible<decltype(&*cct),                         \