Closed brifordwylie closed 3 years ago
🤦
86 warnings remain after PR https://github.com/kbandla/dpkt/pull/521 merge
majority are about imports in dpkt/__init__.py
, to be refactored in https://github.com/kbandla/dpkt/issues/501
13 are code complexity (C901)
./dpkt/http.py:40:1: C901 'parse_body' is too complex (12)
./dpkt/http.py:259:17: F841 local variable 'ret' is assigned to but never used
./dpkt/http.py:318:128: E501 line too long (1358 > 127 characters)
./dpkt/bgp.py:315:13: C901 'BGP.Update.Attribute.unpack' is too complex (14)
./dpkt/ipip.py:5:1: F401 '.ip.IP as IPIP' imported but unused
./dpkt/ieee80211.py:285:5: C901 'IEEE80211.unpack' is too complex (12)
./dpkt/pcapng.py:505:5: C901 'Reader.__init__' is too complex (14)
./dpkt/ethernet.py:82:5: C901 'Ethernet._unpack_data' is too complex (12)
./dpkt/ethernet.py:128:5: C901 'Ethernet.unpack' is too complex (11)
./dpkt/ethernet.py:190:5: C901 'Ethernet.pack_hdr' is too complex (16)
./dpkt/ethernet.py:408:5: F401 '.ip' imported but unused
./dpkt/ethernet.py:670:5: F401 '.ip' imported but unused
./dpkt/gzip.py:76:5: C901 'Gzip.unpack' is too complex (12)
./dpkt/asn1.py:69:1: C901 'decode' is too complex (16)
./dpkt/asn1.py:149:128: E501 line too long (976 > 127 characters)
./dpkt/compat.py:21:5: F401 'io.StringIO' imported but unused
./dpkt/compat.py:26:5: F401 'io.BytesIO' imported but unused
./dpkt/dns.py:255:9: C901 'DNS.RR.pack_rdata' is too complex (12)
./dpkt/dns.py:289:9: C901 'DNS.RR.unpack_rdata' is too complex (13)
./dpkt/__init__.py:12:1: F403 'from .dpkt import *' used; unable to detect undefined names
./dpkt/__init__.py:12:1: F401 '.dpkt.*' imported but unused
./dpkt/__init__.py:14:1: F401 '.ah' imported but unused
./dpkt/__init__.py:15:1: F401 '.aoe' imported but unused
./dpkt/__init__.py:16:1: F401 '.aim' imported but unused
./dpkt/__init__.py:17:1: F401 '.arp' imported but unused
./dpkt/__init__.py:18:1: F401 '.asn1' imported but unused
./dpkt/__init__.py:19:1: F401 '.bgp' imported but unused
./dpkt/__init__.py:20:1: F401 '.cdp' imported but unused
./dpkt/__init__.py:21:1: F401 '.dhcp' imported but unused
./dpkt/__init__.py:22:1: F401 '.diameter' imported but unused
./dpkt/__init__.py:23:1: F401 '.dns' imported but unused
./dpkt/__init__.py:24:1: F401 '.dtp' imported but unused
./dpkt/__init__.py:25:1: F401 '.esp' imported but unused
./dpkt/__init__.py:26:1: F401 '.ethernet' imported but unused
./dpkt/__init__.py:27:1: F401 '.gre' imported but unused
./dpkt/__init__.py:28:1: F401 '.gzip' imported but unused
./dpkt/__init__.py:29:1: F401 '.h225' imported but unused
./dpkt/__init__.py:30:1: F401 '.hsrp' imported but unused
./dpkt/__init__.py:31:1: F401 '.http' imported but unused
./dpkt/__init__.py:32:1: F401 '.http2' imported but unused
./dpkt/__init__.py:33:1: F401 '.icmp' imported but unused
./dpkt/__init__.py:34:1: F401 '.icmp6' imported but unused
./dpkt/__init__.py:35:1: F401 '.ieee80211' imported but unused
./dpkt/__init__.py:36:1: F401 '.igmp' imported but unused
./dpkt/__init__.py:37:1: F401 '.ip' imported but unused
./dpkt/__init__.py:38:1: F401 '.ip6' imported but unused
./dpkt/__init__.py:39:1: F401 '.ipx' imported but unused
./dpkt/__init__.py:40:1: F401 '.llc' imported but unused
./dpkt/__init__.py:41:1: F401 '.loopback' imported but unused
./dpkt/__init__.py:42:1: F401 '.mrt' imported but unused
./dpkt/__init__.py:43:1: F401 '.netbios' imported but unused
./dpkt/__init__.py:44:1: F401 '.netflow' imported but unused
./dpkt/__init__.py:45:1: F401 '.ntp' imported but unused
./dpkt/__init__.py:46:1: F401 '.ospf' imported but unused
./dpkt/__init__.py:47:1: F401 '.pcap' imported but unused
./dpkt/__init__.py:48:1: F401 '.pcapng' imported but unused
./dpkt/__init__.py:49:1: F401 '.pim' imported but unused
./dpkt/__init__.py:50:1: F401 '.pmap' imported but unused
./dpkt/__init__.py:51:1: F401 '.ppp' imported but unused
./dpkt/__init__.py:52:1: F401 '.pppoe' imported but unused
./dpkt/__init__.py:53:1: F401 '.qq' imported but unused
./dpkt/__init__.py:54:1: F401 '.radiotap' imported but unused
./dpkt/__init__.py:55:1: F401 '.radius' imported but unused
./dpkt/__init__.py:56:1: F401 '.rfb' imported but unused
./dpkt/__init__.py:57:1: F401 '.rip' imported but unused
./dpkt/__init__.py:58:1: F401 '.rpc' imported but unused
./dpkt/__init__.py:59:1: F401 '.rtp' imported but unused
./dpkt/__init__.py:60:1: F401 '.rx' imported but unused
./dpkt/__init__.py:61:1: F401 '.sccp' imported but unused
./dpkt/__init__.py:62:1: F401 '.sctp' imported but unused
./dpkt/__init__.py:63:1: F401 '.sip' imported but unused
./dpkt/__init__.py:64:1: F401 '.sll' imported but unused
./dpkt/__init__.py:65:1: F401 '.smb' imported but unused
./dpkt/__init__.py:66:1: F401 '.ssl' imported but unused
./dpkt/__init__.py:67:1: F401 '.stp' imported but unused
./dpkt/__init__.py:68:1: F401 '.stun' imported but unused
./dpkt/__init__.py:69:1: F401 '.tcp' imported but unused
./dpkt/__init__.py:70:1: F401 '.telnet' imported but unused
./dpkt/__init__.py:71:1: F401 '.tftp' imported but unused
./dpkt/__init__.py:72:1: F401 '.tns' imported but unused
./dpkt/__init__.py:73:1: F401 '.tpkt' imported but unused
./dpkt/__init__.py:74:1: F401 '.udp' imported but unused
./dpkt/__init__.py:75:1: F401 '.vrrp' imported but unused
./dpkt/__init__.py:76:1: F401 '.yahoo' imported but unused
./dpkt/telnet.py:35:1: C901 'strip_options' is too complex (11)
./examples/old/ping.py:33:5: C901 'Ping.main' is too complex (12)
13 C901 'decode' is too complex (16)
2 E501 line too long (976 > 127 characters)
69 F401 '.dpkt.*' imported but unused
1 F403 'from .dpkt import *' used; unable to detect undefined names
1 F841 local variable 'ret' is assigned to but never used
86
@obormot looks good thanks for the effort on this, I think what's left over is acceptable for now, closing this ticket
This ticket/issue is related to #512 but this is just general cleanup.