From 6e9cd8821ca028b5bc05c14b0b4e3454036da6da Mon Sep 17 00:00:00 2001 From: Christian Breunig Date: Sun, 21 Apr 2024 10:33:15 +0200 Subject: vyos.utils: T6244: use list to build up result string When handling optional separators rather build up a list and join the list with the requested delimiter to form the resulting human readable time string. --- python/vyos/utils/convert.py | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/python/vyos/utils/convert.py b/python/vyos/utils/convert.py index c02f0071e..da207b9fb 100644 --- a/python/vyos/utils/convert.py +++ b/python/vyos/utils/convert.py @@ -1,4 +1,4 @@ -# Copyright 2023 VyOS maintainers and contributors +# Copyright 2023-2024 VyOS maintainers and contributors # # This library is free software; you can redistribute it and/or # modify it under the terms of the GNU Lesser General Public @@ -23,34 +23,34 @@ def seconds_to_human(s, separator=""): day = 60 * 60 * 24 hour = 60 * 60 - remainder = 0 - result = "" + result = [] + weeks = s // week if weeks > 0: - result = "{0}w".format(weeks) + result.append(f'{weeks}w') s = s % week days = s // day if days > 0: - result = "{0}{1}{2}d".format(result, separator, days) + result.append(f'{days}d') s = s % day hours = s // hour if hours > 0: - result = "{0}{1}{2}h".format(result, separator, hours) + result.append(f'{hours}h') s = s % hour minutes = s // 60 if minutes > 0: - result = "{0}{1}{2}m".format(result, separator, minutes) + result.append(f'{minutes}m') s = s % 60 seconds = s if seconds > 0: - result = "{0}{1}{2}s".format(result, separator, seconds) + result.append(f'{seconds}s') - return result + return separator.join(result) def bytes_to_human(bytes, initial_exponent=0, precision=2, int_below_exponent=0): -- cgit v1.2.3