review comments

Author: dead10ck

helix-core/src/auto_pairs.rs

@@ -272,6 +272,9 @@ fn handle_open(doc: &Rope, selection: &Selection, pair: &Pair) -> Transaction {
         let next_char = doc.get_char(cursor);
         let len_inserted;
 
+        // Since auto pairs are currently limited to single chars, we're either
+        // inserting exactly one or two chars. When arbitrary length pairs are
+        // added, these will need to be changed.
         let change = match next_char {
             Some(_) if !pair.should_close(doc, start_range) => {
                 len_inserted = 1;

helix-term/tests/test/auto_pairs.rs

@@ -28,8 +28,9 @@ async fn insert_basic() -> anyhow::Result<()> {
 
 #[tokio::test]
 async fn insert_configured_multi_byte_chars() -> anyhow::Result<()> {
-    // [NOTE] these are multi-byte Unicode characters
-    let pairs = hashmap!('„' => '“', '‚' => '‘');
+    // NOTE: these are multi-byte Unicode characters
+    let pairs = hashmap!('„' => '“', '‚' => '‘', '「' => '」');
+
     let config = Config {
         editor: helix_view::editor::Config {
             auto_pairs: AutoPairConfig::Pairs(pairs.clone()),