mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexander Rukletsov" <>
Subject Re: Review Request 41609: Cleaned up formatting for multiple inheritance.
Date Mon, 21 Dec 2015 16:56:22 GMT

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

(Updated Dec. 21, 2015, 4:56 p.m.)

Review request for mesos, Bernd Mathiske, Joerg Schad, and Neil Conway.

Repository: mesos


Cleaned up formatting for multiple inheritance.


  src/tests/containerizer/provisioner_docker_tests.cpp 7c1197ed03df8b5d77d26307560535f9ff5bf350

  src/tests/hierarchical_allocator_tests.cpp 88bb7e987c471587d46e15082fe1149fd94db5d6 
  src/tests/registrar_tests.cpp c5022233fa88c4d8578055bc4812055b4cb9060d 


Testing (updated)

None: not a functional change.

I've grepped the code base to find out what is the common wrapping here. 
"tests/executor_http_api_tests.cpp:74" uses the proposed wrapping:
class ExecutorHttpApiTest
  : public MesosTest,
    public WithParamInterface<ContentType> {};

It also seems to be consistent another similar syntax pattern (c-tor initialization list),
for example see


Alexander Rukletsov

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