Commit 6a57d870 authored by Dominik Charousset's avatar Dominik Charousset

Integrate review feedback

parent 8ae37687
...@@ -22,8 +22,6 @@ ...@@ -22,8 +22,6 @@
#include "caf/test/dsl.hpp" #include "caf/test/dsl.hpp"
#include <utility>
#include "caf/event_based_actor.hpp" #include "caf/event_based_actor.hpp"
using namespace caf; using namespace caf;
......
...@@ -106,7 +106,7 @@ public: ...@@ -106,7 +106,7 @@ public:
template <size_t X> template <size_t X>
using pos = std::integral_constant<size_t, X>; using pos = std::integral_constant<size_t, X>;
elementwise_compare_inspector(const Tup& xs) : xs_(xs) { explicit elementwise_compare_inspector(const Tup& xs) : xs_(xs) {
// nop // nop
} }
...@@ -294,7 +294,7 @@ bool received(caf_handle x) { ...@@ -294,7 +294,7 @@ bool received(caf_handle x) {
template <class... Ts> template <class... Ts>
class expect_clause { class expect_clause {
public: public:
expect_clause(caf::scheduler::test_coordinator& sched) explicit expect_clause(caf::scheduler::test_coordinator& sched)
: sched_(sched), dest_(nullptr) { : sched_(sched), dest_(nullptr) {
peek_ = [=] { peek_ = [=] {
/// The extractor will call CAF_FAIL on a type mismatch, essentially /// The extractor will call CAF_FAIL on a type mismatch, essentially
...@@ -372,7 +372,7 @@ protected: ...@@ -372,7 +372,7 @@ protected:
template <> template <>
class expect_clause<void> { class expect_clause<void> {
public: public:
expect_clause(caf::scheduler::test_coordinator& sched) explicit expect_clause(caf::scheduler::test_coordinator& sched)
: sched_(sched), dest_(nullptr) { : sched_(sched), dest_(nullptr) {
// nop // nop
} }
...@@ -431,7 +431,7 @@ protected: ...@@ -431,7 +431,7 @@ protected:
template <class... Ts> template <class... Ts>
class inject_clause { class inject_clause {
public: public:
inject_clause(caf::scheduler::test_coordinator& sched) explicit inject_clause(caf::scheduler::test_coordinator& sched)
: sched_(sched), dest_(nullptr) { : sched_(sched), dest_(nullptr) {
// nop // nop
} }
...@@ -496,7 +496,7 @@ protected: ...@@ -496,7 +496,7 @@ protected:
template <class... Ts> template <class... Ts>
class allow_clause { class allow_clause {
public: public:
allow_clause(caf::scheduler::test_coordinator& sched) explicit allow_clause(caf::scheduler::test_coordinator& sched)
: sched_(sched), dest_(nullptr) { : sched_(sched), dest_(nullptr) {
peek_ = [=] { peek_ = [=] {
if (dest_ != nullptr) if (dest_ != nullptr)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment