From 2cdd884401b7eda29276a1f6c9a80bd0ce89421b Mon Sep 17 00:00:00 2001
From: Turiiya <tobmoj@web.de>
Date: Thu, 11 Apr 2024 16:27:18 +0200
Subject: [PATCH] builtin: cleanup `compare_strings` functions

---
 vlib/builtin/string.v | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/vlib/builtin/string.v b/vlib/builtin/string.v
index 095751b3599496..b02ec549c5d726 100644
--- a/vlib/builtin/string.v
+++ b/vlib/builtin/string.v
@@ -1785,24 +1785,20 @@ pub fn (s string) trim_string_right(str string) string {
 
 // compare_strings returns `-1` if `a < b`, `1` if `a > b` else `0`.
 pub fn compare_strings(a &string, b &string) int {
-	if a < b {
-		return -1
-	}
-	if a > b {
-		return 1
+	return match true {
+		a < b { -1 }
+		a > b { 1 }
+		else { 0 }
 	}
-	return 0
 }
 
 // compare_strings_by_len returns `-1` if `a.len < b.len`, `1` if `a.len > b.len` else `0`.
 fn compare_strings_by_len(a &string, b &string) int {
-	if a.len < b.len {
-		return -1
-	}
-	if a.len > b.len {
-		return 1
+	return match true {
+		a.len < b.len { -1 }
+		a.len > b.len { 1 }
+		else { 0 }
 	}
-	return 0
 }
 
 // compare_lower_strings returns the same as compare_strings but converts `a` and `b` to lower case before comparing.