Просмотр исходного кода

refactor(bridge): avoid unnecessary `maps:to_list/1` when listing

Andrew Mayorov 2 лет назад
Родитель
Сommit
53bc27e0f4
1 измененных файлов с 6 добавлено и 6 удалено
  1. 6 6
      apps/emqx_bridge/src/emqx_bridge.erl

+ 6 - 6
apps/emqx_bridge/src/emqx_bridge.erl

@@ -226,21 +226,21 @@ post_config_update(_, _Req, NewConf, OldConf, _AppEnv) ->
     Result.
 
 list() ->
-    lists:foldl(
-        fun({Type, NameAndConf}, Bridges) ->
-            lists:foldl(
-                fun({Name, RawConf}, Acc) ->
+    maps:fold(
+        fun(Type, NameAndConf, Bridges) ->
+            maps:fold(
+                fun(Name, RawConf, Acc) ->
                     case lookup(Type, Name, RawConf) of
                         {error, not_found} -> Acc;
                         {ok, Res} -> [Res | Acc]
                     end
                 end,
                 Bridges,
-                maps:to_list(NameAndConf)
+                NameAndConf
             )
         end,
         [],
-        maps:to_list(emqx:get_raw_config([bridges], #{}))
+        emqx:get_raw_config([bridges], #{})
     ).
 
 lookup(Id) ->