helix-mirror/helix-view
Michael Davis 98ef05d768 Prefer RopeSlice to &Rope in helix_core::syntax
Pascal and I discussed this and we think it's generally better to
take a 'RopeSlice' rather than a '&Rope'. The code block rendering
function in the markdown component module is a good example for how
this can be useful: we can remove an allocation of a rope and instead
directly turn a '&str' into a 'RopeSlice' which is very cheap.

A change to prefer 'RopeSlice' to '&Rope' whenever the rope isn't
modified would be nice, but it would be a very large diff (around 500+
500-). Starting off with just the syntax functions seems like a nice
middle-ground, and we can remove a Rope allocation because of it.

Co-authored-by: Pascal Kuthe <pascal.kuthe@semimod.de>
2023-07-27 11:50:19 +09:00
..
src Prefer RopeSlice to &Rope in helix_core::syntax 2023-07-27 11:50:19 +09:00
tests/encoding Move helix-view/tests/*txt files to txts subdirectory (#372) 2021-06-25 15:59:06 +09:00
Cargo.toml build(deps): bump once_cell from 1.17.2 to 1.18.0 (#7248) 2023-06-06 11:14:36 +09:00