remove redundant lock calls, use /* unused */

PowerDNS · Nov 29, 2024 · ea9e130 · ea9e130
1 parent b6a8c99
commit ea9e130
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 18 deletions.
diff --git a/pdns/auth-main.cc b/pdns/auth-main.cc
@@ -1210,7 +1210,6 @@ int main(int argc, char** argv)
 {
   versionSetProduct(ProductAuthoritative);
   reportAllTypes(); // init MOADNSParser
-  DNSRecordContent::lock();
 
   g_programname = "pdns";
   g_starttime = time(nullptr);

diff --git a/pdns/dnsrecords.cc b/pdns/dnsrecords.cc
@@ -458,9 +458,8 @@ boilerplate_conv(LP,
                  conv.xfrName(d_fqdn, false);)
 
 /* EUI48 start */
-void EUI48RecordContent::report(const ReportIsOnlyCallableByReportAllTypes& guard)
+void EUI48RecordContent::report(const ReportIsOnlyCallableByReportAllTypes& /* unused */)
 {
-  (void)guard;
   regist(1, QType::EUI48, &make, &make, "EUI48");
 }
 std::shared_ptr<DNSRecordContent> EUI48RecordContent::make(const DNSRecord &dr, PacketReader& pr)
@@ -503,9 +502,8 @@ string EUI48RecordContent::getZoneRepresentation(bool /* noDot */) const
 
 /* EUI64 start */
 
-void EUI64RecordContent::report(const ReportIsOnlyCallableByReportAllTypes& guard)
+void EUI64RecordContent::report(const ReportIsOnlyCallableByReportAllTypes& /* unused */)
 {
-  (void)guard;
   regist(1, QType::EUI64, &make, &make, "EUI64");
 }
 std::shared_ptr<DNSRecordContent> EUI64RecordContent::make(const DNSRecord &dr, PacketReader& pr)
@@ -550,9 +548,8 @@ string EUI64RecordContent::getZoneRepresentation(bool /* noDot */) const
 
 /* APL start */
 /* https://tools.ietf.org/html/rfc3123 */
-void APLRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& guard)
+void APLRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& /* unused */)
 {
-  (void)guard;
   regist(1, QType::APL, &make, &make, "APL");
 }
 

diff --git a/pdns/nsecrecords.cc b/pdns/nsecrecords.cc
@@ -164,9 +164,8 @@ string NSECBitmap::getZoneRepresentation() const
   return ret;
 }
 
-void NSECRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& guard)
+void NSECRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& /* unused */)
 {
-  (void)guard;
   regist(1, 47, &make, &make, "NSEC");
 }
 
@@ -214,9 +213,8 @@ string NSECRecordContent::getZoneRepresentation(bool /* noDot */) const
 
 ////// begin of NSEC3
 
-void NSEC3RecordContent::report(const ReportIsOnlyCallableByReportAllTypes& guard)
+void NSEC3RecordContent::report(const ReportIsOnlyCallableByReportAllTypes& /* unused */)
 {
-  (void)guard;
   regist(1, 50, &make, &make, "NSEC3");
 }
 
@@ -288,9 +286,8 @@ string NSEC3RecordContent::getZoneRepresentation(bool /* noDot */) const
 }
 
 
-void NSEC3PARAMRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& guard)
+void NSEC3PARAMRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& /* unused */)
 {
-  (void)guard;
   regist(1, 51, &make, &make, "NSEC3PARAM");
   regist(254, 51, &make, &make, "NSEC3PARAM");
 }
@@ -346,9 +343,8 @@ string NSEC3PARAMRecordContent::getZoneRepresentation(bool /* noDot */) const
 
 ////// begin of CSYNC
 
-void CSYNCRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& guard)
+void CSYNCRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& /* unused */)
 {
-  (void)guard;
   regist(1, 62, &make, &make, "CSYNC");
 }
 

diff --git a/pdns/recursordist/rec-main.cc b/pdns/recursordist/rec-main.cc
@@ -3154,7 +3154,6 @@ int main(int argc, char** argv)
   g_argv = argv;
   versionSetProduct(ProductRecursor);
   reportAllTypes();
-  DNSRecordContent::lock();
 
   int ret = EXIT_SUCCESS;
 

diff --git a/pdns/sillyrecords.cc b/pdns/sillyrecords.cc
@@ -156,9 +156,8 @@ latlon2ul(const char **latlonstrptr, int *which)
   return (retval);
 }
 
-void LOCRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& guard)
+void LOCRecordContent::report(const ReportIsOnlyCallableByReportAllTypes& /* unused */)
 {
-  (void)guard;
   regist(1, QType::LOC, &make, &make, "LOC");
   regist(254, QType::LOC, &make, &make, "LOC");
 }