-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18823/
-----------------------------------------------------------
(Updated March 5, 2014, 5:59 p.m.)
Review request for mesos.
Changes
-------
Fixed review issues (cpplint.py to go into a different JIRA/review). This time
the patch contains all the *.hpp file changes :-)
Bugs: MESOS-1055
https://issues.apache.org/jira/browse/MESOS-1055
Repository: mesos-git
Description
-------
Put "explicit" in front of all single-argument constructors in C++ Mesos source
code. (Omitted lib process, filed a separate issue for it.)
Diffs (updated)
-----
include/mesos/executor.hpp 7bc8eca
include/mesos/resources.hpp 59f495c
src/common/attributes.hpp a08cf18
src/common/lock.hpp 5af2f42
src/examples/long_lived_framework.cpp dc123cd
src/hdfs/hdfs.hpp 228f007
src/jvm/java/io.hpp fa0c60c
src/jvm/java/lang.hpp 5eb365d
src/jvm/java/net.hpp 7816251
src/jvm/jvm.hpp 2d3a7eb
src/log/log.hpp 628a4b8
src/log/log.cpp 7f855f2
src/log/network.hpp 9c76bf8
src/log/replica.hpp 08ddcb1
src/log/replica.cpp 1f1a945
src/master/allocator.hpp 2e6a910
src/master/contender.hpp 0048ee0
src/master/contender.cpp 42d9c07
src/master/detector.hpp 533027a
src/master/detector.cpp 3a8aaed
src/master/hierarchical_allocator_process.hpp 3ec453a
src/master/master.hpp 49a3e15
src/master/registrar.hpp 20734af
src/master/registrar.cpp 37337c0
src/python/native/proxy_executor.hpp 9f6b33f
src/python/native/proxy_scheduler.hpp cc92149
src/sasl/authenticator.hpp e1db783
src/slave/containerizer/isolator.hpp fc6c9ab
src/slave/monitor.hpp c042bc1
src/slave/slave.hpp 01b80df
src/state/leveldb.hpp 14a94cc
src/state/protobuf.hpp d1c21c6
src/state/state.hpp 133752d
src/state/zookeeper.hpp d1d1fed
src/tests/cgroups_tests.cpp 4a092cc
src/tests/containerizer.hpp 5686398
src/zookeeper/detector.hpp 73235c0
src/zookeeper/detector.cpp e186e51
src/zookeeper/group.hpp d1ead38
src/zookeeper/watcher.hpp 1db0386
Diff: https://reviews.apache.org/r/18823/diff/
Testing
-------
Compiled on Mac OS 10.9 with Clang 3.3 and on Ubuntu 13.10 with gcc 4.7.1. Ran
make check on both.
Also ran: python cpplint.py --filter=-,+runtime/explicit `find mesos -name
'*.cpp'; find mesos -name '*.hpp'`
And due to the patch, no more warnings about explicit constructors inside Mesos
source code.
Note that cpplint.py is now patched to allow '*.hpp' files, which it did not.
Thanks,
Bernd Mathiske
