dnsdist: Use static arrays for the protocols, as suggested by Otto

diff --git a/pdns/dnsdist-protocols.cc b/pdns/dnsdist-protocols.cc
index e6c47a6441183884812823c33e8a11f6a55b10f9..4fc02eeb63b74fedddabbfa8f8e6195239596e93 100644 (file)
--- a/pdns/dnsdist-protocols.cc
+++ b/pdns/dnsdist-protocols.cc
@@ -27,7 +27,7 @@
 
 namespace dnsdist
 {
-const std::vector<std::string> Protocol::s_names = {
+const std::array<std::string, Protocol::s_numberOfProtocols> Protocol::s_names = {
   "DoUDP",
   "DoTCP",
   "DNSCryptUDP",
@@ -35,7 +35,7 @@ const std::vector<std::string> Protocol::s_names = {
   "DoT",
   "DoH"};
 
-static const std::vector<std::string> prettyNames = {
+const std::array<std::string, Protocol::s_numberOfProtocols> Protocol::s_prettyNames = {
   "Do53 UDP",
   "Do53 TCP",
   "DNSCrypt UDP",
@@ -71,7 +71,7 @@ const std::string& Protocol::toString() const
 
 const std::string& Protocol::toPrettyString() const
 {
-  return prettyNames.at(static_cast<uint8_t>(d_protocol));
+  return s_prettyNames.at(static_cast<uint8_t>(d_protocol));
 }
 
 }

diff --git a/pdns/dnsdist-protocols.hh b/pdns/dnsdist-protocols.hh
index 981a8860d460ddef45c861db814b515426483105..fbd51683b815096aecf6b350f0eae4412ec6962a 100644 (file)
--- a/pdns/dnsdist-protocols.hh
+++ b/pdns/dnsdist-protocols.hh
@@ -21,7 +21,7 @@
  */
 #pragma once
 
-#include <vector>
+#include <array>
 #include <string>
 
 namespace dnsdist
@@ -58,6 +58,8 @@ public:
 private:
   typeenum d_protocol;
 
-  static const std::vector<std::string> s_names;
+  static constexpr size_t s_numberOfProtocols = 6;
+  static const std::array<std::string, s_numberOfProtocols> s_names;
+  static const std::array<std::string, s_numberOfProtocols> s_prettyNames;
 };
 }