faster shuffle()

daveoncode · daveoncode · commit d903db3c24de · 2020-03-04T21:03:37.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -86,6 +86,7 @@ signature has changed (now it has two optional boolean arguments: `ignore_spaces
 (it now makes use of the new extracted method `asciify()`)
 - `is_uuid()` has now a second parameter `allow_hex` that if true, considers as valid UUID hex value
 - `uuid()` has now an optional boolean parameter `as_hex` which allows to return UUID string as hex representation
+- `shuffle()` is now faster
 
 ---
 
diff --git a/string_utils/manipulation.py b/string_utils/manipulation.py
@@ -370,7 +370,7 @@ def shuffle(input_string: str) -> str:
         raise InvalidInputError(input_string)
 
     # turn the string into a list of chars
-    chars = sorted(input_string)
+    chars = list(input_string)
 
     # shuffle the list
     random.shuffle(chars)
