From bb199dd3cb8230320bacbee7951fea63ce651733 Mon Sep 17 00:00:00 2001
From: Daniel Salzman <daniel.salzman@nic.cz>
Date: Tue, 24 Jan 2017 20:01:59 +0100
Subject: [PATCH] tests-extra: remove unused variable and early return

---
 tests-extra/tools/dnstest/response.py | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/tests-extra/tools/dnstest/response.py b/tests-extra/tools/dnstest/response.py
index f5ae00d3bb..3c0d6790b8 100644
--- a/tests-extra/tools/dnstest/response.py
+++ b/tests-extra/tools/dnstest/response.py
@@ -208,7 +208,6 @@ class Response(object):
         self.check_xfr()
 
         # 2) Check if Answer contains AXFR data (first SOA, second non-SOA)
-        soa_count = 0
         rr_count = 0
 
         self.resp, iter_copy = itertools.tee(self.resp)
@@ -219,16 +218,10 @@ class Response(object):
                         if rr.rdtype != dns.rdatatype.SOA:
                             set_err("First RR is not SOA")
                             return
-                        else:
-                            soa_count += 1
-
                     elif rr_count == 1:
                         if rr.rdtype == dns.rdatatype.SOA:
                             set_err("Second RR is SOA")
                             return
-                    else:
-                        # OK, it has the format of AXFR
-                        return
 
                     rr_count += 1
 
-- 
GitLab