Commit 10a157f0 authored by Samir Halilcevic's avatar Samir Halilcevic Committed by Dominik Charousset

Fix compiler warnings on Linux

parent 64fa6b01
...@@ -172,7 +172,7 @@ struct variant_inspector_traits<shape_ptr> { ...@@ -172,7 +172,7 @@ struct variant_inspector_traits<shape_ptr> {
// Assigns a value to x. // Assigns a value to x.
template <class U> template <class U>
static void assign(value_type& x, U value) { static void assign(value_type& x, [[maybe_unused]] U value) {
if constexpr (std::is_same_v<U, none_t>) if constexpr (std::is_same_v<U, none_t>)
x.reset(); x.reset();
else else
......
...@@ -32,7 +32,8 @@ public: ...@@ -32,7 +32,8 @@ public:
} }
template <class ErrorPolicy, class TimePoint> template <class ErrorPolicy, class TimePoint>
read_result pull(ErrorPolicy policy, T& item, TimePoint timeout) { read_result
pull(ErrorPolicy policy, T& item, [[maybe_unused]] TimePoint timeout) {
if (!buf_) { if (!buf_) {
return abort_reason_ ? read_result::abort : read_result::stop; return abort_reason_ ? read_result::abort : read_result::stop;
} }
......
...@@ -22,7 +22,8 @@ public: ...@@ -22,7 +22,8 @@ public:
// nop // nop
} }
void actor_exited(const error& fail_state, execution_unit* host) override { void actor_exited(const error& fail_state,
[[maybe_unused]] execution_unit* host) override {
if constexpr (num_args == 0) if constexpr (num_args == 0)
fn_(); fn_();
else if constexpr (num_args == 1) else if constexpr (num_args == 1)
......
...@@ -56,7 +56,7 @@ bool load(deserializer& source, void* ptr) { ...@@ -56,7 +56,7 @@ bool load(deserializer& source, void* ptr) {
} }
template <class T> template <class T>
void stringify(std::string& buf, const void* ptr) { void stringify(std::string& buf, [[maybe_unused]] const void* ptr) {
if constexpr (is_allowed_unsafe_message_type_v<T>) { if constexpr (is_allowed_unsafe_message_type_v<T>) {
auto tn = type_name_v<T>; auto tn = type_name_v<T>;
buf.insert(buf.end(), tn.begin(), tn.end()); buf.insert(buf.end(), tn.begin(), tn.end());
......
...@@ -12,7 +12,7 @@ using namespace caf; ...@@ -12,7 +12,7 @@ using namespace caf;
#define ASSERT_COMPILES(expr, msg) \ #define ASSERT_COMPILES(expr, msg) \
static_assert( \ static_assert( \
std::is_void_v<decltype(std::declval<scheduled_actor*>()->expr)>, msg); std::is_void_v<decltype(std::declval<scheduled_actor*>()->expr)>, msg)
namespace { namespace {
......
...@@ -420,7 +420,7 @@ struct rejecting_fixture { ...@@ -420,7 +420,7 @@ struct rejecting_fixture {
} // namespace } // namespace
BEGIN_FIXTURE_SCOPE(rejecting_fixture); BEGIN_FIXTURE_SCOPE(rejecting_fixture)
SCENARIO("apps can return errors to shut down the framing layer") { SCENARIO("apps can return errors to shut down the framing layer") {
GIVEN("an app that returns -1 for any frame it receives") { GIVEN("an app that returns -1 for any frame it receives") {
...@@ -612,7 +612,7 @@ SCENARIO("the application shuts down on invalid frame fragments") { ...@@ -612,7 +612,7 @@ SCENARIO("the application shuts down on invalid frame fragments") {
} }
} }
END_FIXTURE_SCOPE(); END_FIXTURE_SCOPE()
TEST_CASE("empty closing payload is valid") { TEST_CASE("empty closing payload is valid") {
auto error auto error
......
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