Reverse the polarity, so that it is actually *right*, and make the nearby comment clearer.bug24230

1 files changed, 6 insertions, 3 deletions

diff --git a/src/mirrored_supervisor.erl b/src/mirrored_supervisor.erl
index e273470f..8dfe39f8 100644
--- a/src/mirrored_supervisor.erl
+++ b/src/mirrored_supervisor.erl
@@ -311,9 +311,12 @@ init({overall, Group, Init}) ->
             Mirroring = {mirroring, {?MODULE, start_internal,
                                      [Group, ChildSpecs]},
                          permanent, 16#ffffffff, worker, [?MODULE]},
-            %% Important: Delegate MUST start after Mirroring, see comment in
-            %% handle_info('DOWN', ...) below
-            {ok, {{one_for_all, 0, 1}, [Mirroring, Delegate]}};
+            %% Important: Delegate MUST start before Mirroring so that
+            %% when we shut down from above it shuts down last, so
+            %% Mirroring does not see it die.
+            %%
+            %% See comment in handle_info('DOWN', ...) below
+            {ok, {{one_for_all, 0, 1}, [Delegate, Mirroring]}};
         ignore ->
             ignore
     end;