[clang-tidy] add readability-redundant-control-flow (#8365)

script/make-pretty

@@ -144,6 +144,7 @@ readability-avoid-const-params-in-decls,\
 readability-else-after-return,\
 readability-inconsistent-declaration-parameter-name,\
 readability-make-member-function-const,\
+readability-redundant-control-flow,\
 readability-redundant-member-init,\
 readability-simplify-boolean-expr,\
 readability-static-accessed-through-instance,\

src/core/thread/dua_manager.cpp

@@ -767,8 +767,6 @@ void DuaManager::UpdateChildDomainUnicastAddress(const Child &aChild, Mle::Child
         mChildDuaMask.Set(childIndex, true);
         mChildDuaRegisteredMask.Set(childIndex, false);
     }
-
-    return;
 }
 #endif // OPENTHREAD_FTD && OPENTHREAD_CONFIG_TMF_PROXY_DUA_ENABLE
 

src/core/thread/link_metrics.cpp

@@ -499,7 +499,6 @@ void LinkMetrics::HandleReport(const Message &     aMessage,
 
 exit:
     LogDebg("HandleReport, error:%s", ErrorToString(error));
-    return;
 }
 
 Error LinkMetrics::HandleLinkProbe(const Message &aMessage, uint8_t &aSeriesId)

src/core/thread/network_diagnostic.cpp

@@ -163,7 +163,6 @@ exit:
     {
         LogDebg("Received diagnostic get response, error = %s", ErrorToString(aResult));
     }
-    return;
 }
 
 template <>