[Git][NTPsec/ntpsec][master] Removed unused argument idx for ControlPacket.stats()

[Ian Bruene]

Ian Bruene pushed to branch master at NTPsec / ntpsec


Commits:
583291a0 by Ian Bruene at 2017-08-23T15:36:01-05:00
Removed unused argument idx for ControlPacket.stats()

Also removed argument passed in call to stats

- - - - -


1 changed file:

- pylib/packet.py


Changes:

=====================================
pylib/packet.py
=====================================
--- a/pylib/packet.py
+++ b/pylib/packet.py
@@ -608,9 +608,8 @@ class ControlPacket(Packet):
     def end(self):
         return self.count + self.offset
 
-    def stats(self, idx=None):
+    def stats(self):
         "Return statistics on a fragment."
-        # idx is default None until I can confirm that it is useless
         return "%5d %5d\t%3d octets\n" % (self.offset, self.end(), self.count)
 
     def analyze(self, rawdata):
@@ -1062,7 +1061,7 @@ class ControlSession:
                         self.logfp.write(
                             "ERR_INCOMPLETE: Received fragments:\n")
                         for (i, frag) in enumerate(fragments):
-                            self.logfp.write("%d: %s" % (i+1, frag.stats(i)))
+                            self.logfp.write("%d: %s" % (i+1, frag.stats()))
                         self.logfp.write("last fragment %sreceived\n"
                                          % ("not ", "")[seenlastfrag])
                 raise ControlException(SERR_INCOMPLETE)



View it on GitLab: https://gitlab.com/NTPsec/ntpsec/commit/583291a07a4e2015bad84acf605b5b139ef0040e

---
View it on GitLab: https://gitlab.com/NTPsec/ntpsec/commit/583291a07a4e2015bad84acf605b5b139ef0040e
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ntpsec.org/pipermail/vc/attachments/20170823/f37d8466/attachment.html>