From e72e49b29183e9c0285f38c500f0ca5b9fdc8550 Mon Sep 17 00:00:00 2001
From: JianBo He <heeejianbo@gmail.com>
Date: Wed, 9 Mar 2022 13:44:53 +0800
Subject: [PATCH] chore: fix elvis warnings

---
 apps/emqx_gateway/src/emqx_gateway_cm.erl   | 6 ++++--
 apps/emqx_gateway/src/emqx_gateway_http.erl | 6 ++++--
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/apps/emqx_gateway/src/emqx_gateway_cm.erl b/apps/emqx_gateway/src/emqx_gateway_cm.erl
index 92fdfd9e5..0e33793f8 100644
--- a/apps/emqx_gateway/src/emqx_gateway_cm.erl
+++ b/apps/emqx_gateway/src/emqx_gateway_cm.erl
@@ -103,6 +103,8 @@
 -define(T_TAKEOVER, 15000).
 -define(DEFAULT_BATCH_SIZE, 10000).
 
+-elvis([{elvis_style, invalid_dynamic_call, disable}]).
+
 %%--------------------------------------------------------------------
 %% APIs
 %%--------------------------------------------------------------------
@@ -391,7 +393,7 @@ takeover_session(GwName, ClientId) ->
         [ChanPid] ->
             do_takeover_session(GwName, ClientId, ChanPid);
         ChanPids ->
-            [ChanPid|StalePids] = lists:reverse(ChanPids),
+            [ChanPid | StalePids] = lists:reverse(ChanPids),
             ?SLOG(warning, #{ msg => "more_than_one_channel_found"
                             , chan_pids => ChanPids
                             }),
@@ -656,7 +658,7 @@ init(Options) ->
     TabOpts = [public, {write_concurrency, true}],
 
     {ChanTab, ConnTab, InfoTab} = cmtabs(GwName),
-    ok = emqx_tables:new(ChanTab, [bag, {read_concurrency, true}|TabOpts]),
+    ok = emqx_tables:new(ChanTab, [bag, {read_concurrency, true} | TabOpts]),
     ok = emqx_tables:new(ConnTab, [bag | TabOpts]),
     ok = emqx_tables:new(InfoTab, [set, compressed | TabOpts]),
 
diff --git a/apps/emqx_gateway/src/emqx_gateway_http.erl b/apps/emqx_gateway/src/emqx_gateway_http.erl
index 4802aa6ff..e0795c950 100644
--- a/apps/emqx_gateway/src/emqx_gateway_http.erl
+++ b/apps/emqx_gateway/src/emqx_gateway_http.erl
@@ -74,8 +74,10 @@
          , listeners => []
          }.
 
--elvis([{elvis_style, god_modules, disable}]).
--elvis([{elvis_style, no_nested_try_catch, disable}]).
+-elvis([ {elvis_style, god_modules, disable}
+       , {elvis_style, no_nested_try_catch, disable}
+       , {elvis_style, invalid_dynamic_call, disable}
+       ]).
 
 -define(DEFAULT_CALL_TIMEOUT, 15000).