Don't calculate symbol width twice

This is potentially costly so we should avoid calling width()
helix-editor · Jan 23, 2022 · ac81b47 · ac81b47
1 parent 66a8612
commit ac81b47
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/helix-tui/src/buffer.rs b/helix-tui/src/buffer.rs
@@ -535,9 +535,10 @@ impl Buffer {
                 updates.push((x, y, &next_buffer[i]));
             }
 
-            to_skip = current.symbol.width().saturating_sub(1);
+            let current_width = current.symbol.width();
+            to_skip = current_width.saturating_sub(1);
 
-            let affected_width = std::cmp::max(current.symbol.width(), previous.symbol.width());
+            let affected_width = std::cmp::max(current_width, previous.symbol.width());
             invalidated = std::cmp::max(affected_width, invalidated).saturating_sub(1);
         }
         updates