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:
parent
5dc25de668
commit
03f638a8ef
|
@ -63,7 +63,7 @@ def get_Kstat():
|
|||
del kstats[0:2]
|
||||
for kstat in kstats:
|
||||
kstat = kstat.strip()
|
||||
name, unused, value = kstat.split()
|
||||
name, _, value = kstat.split()
|
||||
Kstat[namespace + name] = D(value)
|
||||
|
||||
Kstat = {}
|
||||
|
@ -77,17 +77,6 @@ def get_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):
|
||||
"""Return human-readable representation of a byte value in
|
||||
powers of 2 (eg "KiB" for "kibibytes", etc) to two decimal
|
||||
|
@ -908,11 +897,13 @@ unSub = [
|
|||
|
||||
|
||||
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("ZFS Subsystem Report\t\t\t\t%s" % daydate)
|
||||
div2()
|
||||
sys.stdout.write('\n'+'-'*72+'\n')
|
||||
sys.stdout.write('ZFS Subsystem Report\t\t\t\t%s' % daydate)
|
||||
sys.stdout.write('\n')
|
||||
|
||||
|
||||
def usage():
|
||||
|
@ -975,7 +966,7 @@ def main():
|
|||
zfs_header()
|
||||
for page in pages:
|
||||
page(Kstat)
|
||||
div2()
|
||||
sys.stdout.write("\n")
|
||||
|
||||
|
||||
if __name__ == '__main__':
|
||||
|
|
Loading…
Reference in New Issue