mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Neil Conway <>
Subject Review Request 53387: Silenced some `clang-tidy` warnings in test cases.
Date Wed, 02 Nov 2016 16:24:29 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos and Joseph Wu.

Repository: mesos


`clang-tidy` points out, rightly, that an `ASSERT` failure can result in
leaking some heap-allocated values that aren't wrapped in a smart
pointer. This commit fixes the cases that `clang-tidy` complains about
by wrapping the values in `Owned<T>`.

Note that there are many other places in the tests that leak resources
if an exception occurs. The proper fix is usually to use a smart pointer
rather than a raw pointer. However, this is not always easy/clean, in
part because the current `Owned<T>` and `Shared<T>` types do not support
inheritance (MESOS-6496). So for now, just fix the cases that clang-tidy
complains about.


  src/tests/containerizer/mesos_containerizer_tests.cpp 4df537747d84daa68c29e2d05b22fa386a4a16db



`make check`

Verified that observed `clang-tidy` warnings go away with this change.


Neil Conway

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message