Minor code cleanup in arc_summary.py

Simplify and inline single-use function div1(); inline twice-used
function div2(); add function comment to zfs_header(); replace
variable "unused" in get_Kstat() with "_" following convention.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: George Melikov <mail@gmelikov.ru>
Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Signed-off-by: Scot W. Stevenson <scot.stevenson@gmail.com>
Closes #6802
This commit is contained in:
Scot W. Stevenson 2017-11-03 23:43:53 +01:00 committed by Brian Behlendorf
parent 34c2b3680b
commit df1f129bc4

View File

@ -63,7 +63,7 @@ def get_Kstat():
del kstats[0:2] del kstats[0:2]
for kstat in kstats: for kstat in kstats:
kstat = kstat.strip() kstat = kstat.strip()
name, unused, value = kstat.split() name, _, value = kstat.split()
Kstat[namespace + name] = D(value) Kstat[namespace + name] = D(value)
Kstat = {} Kstat = {}
@ -77,17 +77,6 @@ def get_Kstat():
return Kstat return Kstat
def div1():
sys.stdout.write("\n")
for i in range(18):
sys.stdout.write("%s" % "----")
sys.stdout.write("\n")
def div2():
sys.stdout.write("\n")
def fBytes(b=0): def fBytes(b=0):
"""Return human-readable representation of a byte value in """Return human-readable representation of a byte value in
powers of 2 (eg "KiB" for "kibibytes", etc) to two decimal powers of 2 (eg "KiB" for "kibibytes", etc) to two decimal
@ -908,11 +897,13 @@ unSub = [
def zfs_header(): def zfs_header():
daydate = time.strftime("%a %b %d %H:%M:%S %Y") """Print title string with date
"""
daydate = time.strftime('%a %b %d %H:%M:%S %Y')
div1() sys.stdout.write('\n'+'-'*72+'\n')
sys.stdout.write("ZFS Subsystem Report\t\t\t\t%s" % daydate) sys.stdout.write('ZFS Subsystem Report\t\t\t\t%s' % daydate)
div2() sys.stdout.write('\n')
def usage(): def usage():
@ -975,7 +966,7 @@ def main():
zfs_header() zfs_header()
for page in pages: for page in pages:
page(Kstat) page(Kstat)
div2() sys.stdout.write("\n")
if __name__ == '__main__': if __name__ == '__main__':