diff options
author | Ben Hood <0x6e6562@gmail.com> | 2008-09-27 21:51:23 +0100 |
---|---|---|
committer | Ben Hood <0x6e6562@gmail.com> | 2008-09-27 21:51:23 +0100 |
commit | 733ac828df24d7ddcbc534e24627128df565a610 (patch) | |
tree | e7f1a7e18f9e3a82e0f40ceb04815421184ef254 | |
parent | f062146dc1dcf8c148162b7db1febdd711a89129 (diff) | |
download | rabbitmq-server-733ac828df24d7ddcbc534e24627128df565a610.tar.gz |
Removed e_m_t/2
-rw-r--r-- | include/rabbit.hrl | 1 | ||||
-rw-r--r-- | src/rabbit_exchange.erl | 5 | ||||
-rw-r--r-- | src/rabbit_misc.erl | 10 |
3 files changed, 6 insertions, 10 deletions
diff --git a/include/rabbit.hrl b/include/rabbit.hrl index 20a64367..00dc8b97 100644 --- a/include/rabbit.hrl +++ b/include/rabbit.hrl @@ -70,7 +70,6 @@ -type(node() :: atom()). -type(socket() :: port()). -type(thunk(T) :: fun(() -> T)). --type(function(T) :: fun((T) -> T)). %% this is really an abstract type, but dialyzer does not support them -type(guid() :: any()). diff --git a/src/rabbit_exchange.erl b/src/rabbit_exchange.erl index ac0c8e72..22d856da 100644 --- a/src/rabbit_exchange.erl +++ b/src/rabbit_exchange.erl @@ -262,13 +262,14 @@ add_binding(ExchangeName, QueueName, RoutingKey, _Arguments) -> Binding = #binding{exchange_name = ExchangeName, key = RoutingKey, queue_name = QueueName}, - rabbit_misc:execute_mnesia_transaction(fun add_binding/1, [Binding]). + rabbit_misc:execute_mnesia_transaction(fun() -> add_binding(Binding) end). delete_binding(ExchangeName, QueueName, RoutingKey, _Arguments) -> Binding = #binding{exchange_name = ExchangeName, key = RoutingKey, queue_name = QueueName}, - rabbit_misc:execute_mnesia_transaction(fun delete_binding/1, [Binding]). + rabbit_misc:execute_mnesia_transaction(fun() -> delete_binding(Binding) + end). % Must be called from within a transaction add_binding(Binding) -> diff --git a/src/rabbit_misc.erl b/src/rabbit_misc.erl index 1c9d299f..e432db59 100644 --- a/src/rabbit_misc.erl +++ b/src/rabbit_misc.erl @@ -36,7 +36,7 @@ -export([enable_cover/0, report_cover/0]). -export([with_exit_handler/2]). -export([with_user/2, with_vhost/2, with_user_and_vhost/3]). --export([execute_mnesia_transaction/1, execute_mnesia_transaction/2]). +-export([execute_mnesia_transaction/1]). -export([ensure_ok/2]). -export([localnode/1, tcp_name/3]). -export([intersperse/2, upmap/2, map_in_order/2]). @@ -81,8 +81,7 @@ -spec(with_user/2 :: (username(), thunk(A)) -> A). -spec(with_vhost/2 :: (vhost(), thunk(A)) -> A). -spec(with_user_and_vhost/3 :: (username(), vhost(), thunk(A)) -> A). --spec(execute_mnesia_transaction/1 :: (thunk(A) | function(A)) -> A). --spec(execute_mnesia_transaction/2 :: (thunk(A) | function(A), list()) -> A). +-spec(execute_mnesia_transaction/1 :: (thunk(A)) -> A). -spec(ensure_ok/2 :: ('ok' | {'error', any()}, atom()) -> 'ok'). -spec(localnode/1 :: (atom()) -> node()). -spec(tcp_name/3 :: (atom(), ip_address(), ip_port()) -> atom()). @@ -234,10 +233,7 @@ with_user_and_vhost(Username, VHostPath, Thunk) -> %% elsewhere and get a consistent result even when that read %% executes on a different node. execute_mnesia_transaction(TxFun) -> - execute_mnesia_transaction(TxFun, []). - -execute_mnesia_transaction(TxFun, Args) -> - case mnesia:sync_transaction(TxFun, Args) of + case mnesia:sync_transaction(TxFun) of {atomic, Result} -> Result; {aborted, Reason} -> throw({error, Reason}) end. |