From e3ad9d097ea2f7333e5d84f4690528c74149ed94 Mon Sep 17 00:00:00 2001 From: klensy Date: Sun, 14 Jan 2024 15:16:23 +0300 Subject: [PATCH] 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 --- src/renderer/html_handlebars/hbs_renderer.rs | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/renderer/html_handlebars/hbs_renderer.rs b/src/renderer/html_handlebars/hbs_renderer.rs index c701729f..b706108e 100644 --- a/src/renderer/html_handlebars/hbs_renderer.rs +++ b/src/renderer/html_handlebars/hbs_renderer.rs @@ -935,8 +935,9 @@ fn add_playground_pre( /// Modifies all `` blocks to convert "hidden" lines and to wrap them in /// a ``. fn hide_lines(html: &str, code_config: &Code) -> String { - let language_regex = Regex::new(r"\blanguage-(\w+)\b").unwrap(); - let hidelines_regex = Regex::new(r"\bhidelines=(\S+)").unwrap(); + static LANGUAGE_REGEX: Lazy = Lazy::new(|| Regex::new(r"\blanguage-(\w+)\b").unwrap()); + static HIDELINES_REGEX: Lazy = Lazy::new(|| Regex::new(r"\bhidelines=(\S+)").unwrap()); + CODE_BLOCK_RE .replace_all(html, |caps: &Captures<'_>| { let text = &caps[1]; @@ -951,12 +952,12 @@ fn hide_lines(html: &str, code_config: &Code) -> String { ) } else { // First try to get the prefix from the code block - let hidelines_capture = hidelines_regex.captures(classes); + let hidelines_capture = HIDELINES_REGEX.captures(classes); let hidelines_prefix = match &hidelines_capture { Some(capture) => Some(&capture[1]), None => { // Then look up the prefix by language - language_regex.captures(classes).and_then(|capture| { + LANGUAGE_REGEX.captures(classes).and_then(|capture| { code_config.hidelines.get(&capture[1]).map(|p| p.as_str()) }) }