[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v6 5/5] Cleanup: grub-mkconfig_lib: remove unused version compari
From: |
Mathieu Desnoyers |
Subject: |
[PATCH v6 5/5] Cleanup: grub-mkconfig_lib: remove unused version comparison functions |
Date: |
Mon, 13 Jun 2022 10:08:26 -0400 |
There are no users left of version_find_latest(), version_test_gt(), and
version_test_numeric(). Remove those unused helper functions. Using
those helper functions is what caused the quadratic sorting performance
issues in the first place, so removing them is a net win.
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Reviewed-by: Robbie Harwood <rharwood@redhat.com>
---
util/grub-mkconfig_lib.in | 51 ---------------------------------------
1 file changed, 51 deletions(-)
diff --git a/util/grub-mkconfig_lib.in b/util/grub-mkconfig_lib.in
index fa3de6008..634bc8a50 100644
--- a/util/grub-mkconfig_lib.in
+++ b/util/grub-mkconfig_lib.in
@@ -218,57 +218,6 @@ version_sort ()
esac
}
-version_test_numeric ()
-{
- version_test_numeric_a="$1"
- version_test_numeric_cmp="$2"
- version_test_numeric_b="$3"
- if [ "$version_test_numeric_a" = "$version_test_numeric_b" ] ; then
- case "$version_test_numeric_cmp" in
- ge|eq|le) return 0 ;;
- gt|lt) return 1 ;;
- esac
- fi
- if [ "$version_test_numeric_cmp" = "lt" ] ; then
- version_test_numeric_c="$version_test_numeric_a"
- version_test_numeric_a="$version_test_numeric_b"
- version_test_numeric_b="$version_test_numeric_c"
- fi
- if (echo "$version_test_numeric_a" ; echo "$version_test_numeric_b") |
version_sort | head -n 1 | grep -qx "$version_test_numeric_b" ; then
- return 0
- else
- return 1
- fi
-}
-
-version_test_gt ()
-{
- version_test_gt_a="`echo "$1" | sed -e "s/[^-]*-//"`"
- version_test_gt_b="`echo "$2" | sed -e "s/[^-]*-//"`"
- version_test_gt_cmp=gt
- if [ "x$version_test_gt_b" = "x" ] ; then
- return 0
- fi
- case "$version_test_gt_a:$version_test_gt_b" in
- *.old:*.old) ;;
- *.old:*) version_test_gt_a="`echo "$version_test_gt_a" | sed -e
's/\.old$//'`" ; version_test_gt_cmp=gt ;;
- *:*.old) version_test_gt_b="`echo "$version_test_gt_b" | sed -e
's/\.old$//'`" ; version_test_gt_cmp=ge ;;
- esac
- version_test_numeric "$version_test_gt_a" "$version_test_gt_cmp"
"$version_test_gt_b"
- return "$?"
-}
-
-version_find_latest ()
-{
- version_find_latest_a=""
- for i in "$@" ; do
- if version_test_gt "$i" "$version_find_latest_a" ; then
- version_find_latest_a="$i"
- fi
- done
- echo "$version_find_latest_a"
-}
-
# One layer of quotation is eaten by "" and the second by sed; so this turns
# ' into \'.
grub_quote () {
--
2.30.2
- [PATCH v6 0/5] grub-mkconfig: Fix quadratic algorithm for sorting menu items, Mathieu Desnoyers, 2022/06/13
- [PATCH v6 2/5] grub-mkconfig linux_xen: Fix quadratic algorithm for sorting menu items, Mathieu Desnoyers, 2022/06/13
- [PATCH v6 5/5] Cleanup: grub-mkconfig_lib: remove unused version comparison functions,
Mathieu Desnoyers <=
- [PATCH v6 4/5] grub-mkconfig kfreebsd: Fix quadratic algorithm for sorting menu items, Mathieu Desnoyers, 2022/06/13
- [PATCH v6 1/5] grub-mkconfig linux: Fix quadratic algorithm for sorting menu items, Mathieu Desnoyers, 2022/06/13
- [PATCH v6 3/5] grub-mkconfig hurd: Fix quadratic algorithm for sorting menu items, Mathieu Desnoyers, 2022/06/13
- Re: [PATCH v6 0/5] grub-mkconfig: Fix quadratic algorithm for sorting menu items, Daniel Kiper, 2022/06/24