Skip to content

Commit e3ad9d0

Browse files
committed
reduce useless regex allocs
from 474mb to 215mb ==40876== Total: 474,156,323 bytes in 1,521,025 blocks ==40876== At t-gmax: 13,872,954 bytes in 4,655 blocks ==40876== At t-end: 488,516 bytes in 884 blocks ==40876== Reads: 820,933,434 bytes ==40876== Writes: 514,838,350 bytes to ==57763== Total: 215,292,393 bytes in 1,161,048 blocks ==57763== At t-gmax: 13,872,954 bytes in 4,655 blocks ==57763== At t-end: 1,210,783 bytes in 1,274 blocks ==57763== Reads: 598,542,892 bytes ==57763== Writes: 229,841,910 bytes
1 parent 573b652 commit e3ad9d0

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/renderer/html_handlebars/hbs_renderer.rs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -935,8 +935,9 @@ fn add_playground_pre(
935935
/// Modifies all `<code>` blocks to convert "hidden" lines and to wrap them in
936936
/// a `<span class="boring">`.
937937
fn hide_lines(html: &str, code_config: &Code) -> String {
938-
let language_regex = Regex::new(r"\blanguage-(\w+)\b").unwrap();
939-
let hidelines_regex = Regex::new(r"\bhidelines=(\S+)").unwrap();
938+
static LANGUAGE_REGEX: Lazy<Regex> = Lazy::new(|| Regex::new(r"\blanguage-(\w+)\b").unwrap());
939+
static HIDELINES_REGEX: Lazy<Regex> = Lazy::new(|| Regex::new(r"\bhidelines=(\S+)").unwrap());
940+
940941
CODE_BLOCK_RE
941942
.replace_all(html, |caps: &Captures<'_>| {
942943
let text = &caps[1];
@@ -951,12 +952,12 @@ fn hide_lines(html: &str, code_config: &Code) -> String {
951952
)
952953
} else {
953954
// First try to get the prefix from the code block
954-
let hidelines_capture = hidelines_regex.captures(classes);
955+
let hidelines_capture = HIDELINES_REGEX.captures(classes);
955956
let hidelines_prefix = match &hidelines_capture {
956957
Some(capture) => Some(&capture[1]),
957958
None => {
958959
// Then look up the prefix by language
959-
language_regex.captures(classes).and_then(|capture| {
960+
LANGUAGE_REGEX.captures(classes).and_then(|capture| {
960961
code_config.hidelines.get(&capture[1]).map(|p| p.as_str())
961962
})
962963
}

0 commit comments

Comments
 (0)