diff options
author | Jean-Sébastien Pédron <jean-sebastien.pedron@dumbbell.fr> | 2018-02-01 11:57:23 +0100 |
---|---|---|
committer | Jean-Sébastien Pédron <jean-sebastien.pedron@dumbbell.fr> | 2018-02-01 13:16:59 +0100 |
commit | 1032d8551c614d0892f074e3acaf40786622af81 (patch) | |
tree | 0242bfa9662ffdda7023f7d6327f4682609d62a1 | |
parent | 55a794d357b4918c2a20f7dbadcdd7e3257c9a60 (diff) | |
download | rabbitmq-server-git-1032d8551c614d0892f074e3acaf40786622af81.tar.gz |
rabbit_heartbeat, rabbit_writer: system_terminate() doesn't return
Thus, use the correct return type of `no_return()`. Even if it's defined
as `none()` according to the documentation, it doesn't have the same
semantic.
The warning was reported by Dialyzer.
[#153850881]
-rw-r--r-- | deps/rabbit_common/src/rabbit_heartbeat.erl | 2 | ||||
-rw-r--r-- | deps/rabbit_common/src/rabbit_writer.erl | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/deps/rabbit_common/src/rabbit_heartbeat.erl b/deps/rabbit_common/src/rabbit_heartbeat.erl index ac7870236b..8e5c5fcf3b 100644 --- a/deps/rabbit_common/src/rabbit_heartbeat.erl +++ b/deps/rabbit_common/src/rabbit_heartbeat.erl @@ -57,7 +57,7 @@ -spec system_code_change(_,_,_,_) -> {'ok',_}. -spec system_continue(_,_,{_, _}) -> any(). --spec system_terminate(_,_,_,_) -> none(). +-spec system_terminate(_,_,_,_) -> no_return(). %%---------------------------------------------------------------------------- start(SupPid, Sock, SendTimeoutSec, SendFun, ReceiveTimeoutSec, ReceiveFun) -> diff --git a/deps/rabbit_common/src/rabbit_writer.erl b/deps/rabbit_common/src/rabbit_writer.erl index 66f87c32b3..5223071287 100644 --- a/deps/rabbit_common/src/rabbit_writer.erl +++ b/deps/rabbit_common/src/rabbit_writer.erl @@ -96,7 +96,7 @@ -spec system_code_change(_,_,_,_) -> {'ok',_}. -spec system_continue(_,_,#wstate{}) -> any(). --spec system_terminate(_,_,_,_) -> none(). +-spec system_terminate(_,_,_,_) -> no_return(). -spec send_command(pid(), rabbit_framing:amqp_method_record()) -> 'ok'. -spec send_command |