summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBen Hood <0x6e6562@gmail.com>2008-09-21 14:18:21 +0100
committerBen Hood <0x6e6562@gmail.com>2008-09-21 14:18:21 +0100
commit9f55ecc6934d340e53b557a9769f623f8f441992 (patch)
tree32bbec165a885b3c1bf43c1f44fe3ff83c1d13b1
parentbca6c36217ae8a356f23ec603f221ae904187b9e (diff)
downloadrabbitmq-server-git-9f55ecc6934d340e53b557a9769f623f8f441992.tar.gz
Replaced mnesia:delete/1 with mnesia:delete_object/3
-rw-r--r--src/rabbit_exchange.erl12
1 files changed, 5 insertions, 7 deletions
diff --git a/src/rabbit_exchange.erl b/src/rabbit_exchange.erl
index 143fa48d01..63c7d7a113 100644
--- a/src/rabbit_exchange.erl
+++ b/src/rabbit_exchange.erl
@@ -281,8 +281,7 @@ add_binding(Binding) ->
fun (X, Q) -> if Q#amqqueue.durable and not(X#exchange.durable) ->
{error, durability_settings_incompatible};
true ->
- ok = sync_binding(Binding, Q#amqqueue.durable,
- fun mnesia:write/1, fun mnesia:write/3)
+ ok = sync_binding(Binding, Q#amqqueue.durable, fun mnesia:write/3)
end
end).
@@ -290,8 +289,7 @@ add_binding(Binding) ->
delete_binding(Binding) ->
call_with_exchange_and_queue(
Binding,
- fun (X, Q) -> ok = sync_binding(Binding, Q#amqqueue.durable,
- fun mnesia:delete/1, fun mnesia:delete/3),
+ fun (X, Q) -> ok = sync_binding(Binding, Q#amqqueue.durable, fun mnesia:delete_object/3),
maybe_auto_delete(X)
end).
@@ -311,12 +309,12 @@ reverse_binding(#binding{exchange_name = Exchange, key = Key, queue_name = Queue
#reverse_binding{exchange_name = Exchange, key = Key, queue_name = Queue}.
%% Must run within a transaction.
-sync_binding(Binding, Durable, RouteSyncFun, DurableRouteSyncFun) ->
+sync_binding(Binding, Durable, Fun) ->
ok = case Durable of
- true -> DurableRouteSyncFun(durable_routes, #route{binding = Binding}, write);
+ true -> Fun(durable_routes, #route{binding = Binding}, write);
false -> ok
end,
- [ok, ok] = [RouteSyncFun(R) || R <- tuple_to_list(route_with_reverse(Binding))],
+ [ok, ok] = [Fun(element(1, R), R, write) || R <- tuple_to_list(route_with_reverse(Binding))],
ok.
route_with_reverse(#route{binding = Binding}) ->