review comments

spec.md

@@ -38,7 +38,7 @@ Make email addresses clickable, opens the chat with that contact and creates it
 #### Format
 
 - format should follow standards (as time of writing the current implementation is still fairly dumb)
-- trailing `.` is not part of the email address and should not be parsed with it.
+- `.`-s are a part of the email address, but trailing ones are removed.
 
 <a name="mentions" id="mentions"></a>
 

src/parser/parse_from_text/mod.rs

@@ -100,7 +100,7 @@ pub(crate) fn extract_mention_addresses(input: &str) -> Vec<String> {
             break;
         }
     }
-    result.sort();
+    result.sort_unstable();
     result.dedup();
     result
 }

tests/text_to_ast/text_only.rs

@@ -302,56 +302,18 @@ fn mention_do_not_parse_last_dot() {
 
 #[test]
 fn mention_do_not_parse_last_char_if_special() {
-    assert_eq!(
-        parse_only_text("you can ping me via @[email protected]!"),
-        vec![
-            Text("you can ping me via "),
-            Mention {
-                address: "[email protected]"
-            },
-            Text("!")
-        ]
-    );
-    assert_eq!(
-        parse_only_text("you can ping me via @[email protected]?"),
-        vec![
-            Text("you can ping me via "),
-            Mention {
-                address: "[email protected]"
-            },
-            Text("?")
-        ]
-    );
-    assert_eq!(
-        parse_only_text("you can ping me via @[email protected],"),
-        vec![
-            Text("you can ping me via "),
-            Mention {
-                address: "[email protected]"
-            },
-            Text(",")
-        ]
-    );
-    assert_eq!(
-        parse_only_text("you can ping me via @[email protected]:"),
-        vec![
-            Text("you can ping me via "),
-            Mention {
-                address: "[email protected]"
-            },
-            Text(":")
-        ]
-    );
-    assert_eq!(
-        parse_only_text("you can ping me via @[email protected];"),
-        vec![
-            Text("you can ping me via "),
-            Mention {
-                address: "[email protected]"
-            },
-            Text(";")
-        ]
-    );
+    for c in ["!", "?", ",", ":", ";"] {
+        assert_eq!(
+            parse_only_text(&("you can ping me via @[email protected]".to_string() + c)),
+            vec![
+                Text("you can ping me via "),
+                Mention {
+                    address: "[email protected]"
+                },
+                Text(c)
+            ]
+        );
+    }
 }
 
 #[test]