From 573b6522f9eb101cee93989ac3788a73e1e62c5d Mon Sep 17 00:00:00 2001 From: klensy Date: Sun, 14 Jan 2024 14:50:19 +0300 Subject: [PATCH 1/2] remove useless alloc on rust reference book this reduces total allocs from 490mb to 474mb: ==23272== Total: 490,538,699 bytes in 1,760,117 blocks ==23272== At t-gmax: 13,872,954 bytes in 4,655 blocks ==23272== At t-end: 488,516 bytes in 884 blocks ==23272== Reads: 830,509,060 bytes ==23272== Writes: 522,290,614 bytes to ==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 --- src/renderer/html_handlebars/helpers/navigation.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/renderer/html_handlebars/helpers/navigation.rs b/src/renderer/html_handlebars/helpers/navigation.rs index 86b240ba..12c69027 100644 --- a/src/renderer/html_handlebars/helpers/navigation.rs +++ b/src/renderer/html_handlebars/helpers/navigation.rs @@ -103,7 +103,7 @@ fn find_chapter( } } - previous = Some(item.clone()); + previous = Some(item); } _ => continue, } From e3ad9d097ea2f7333e5d84f4690528c74149ed94 Mon Sep 17 00:00:00 2001 From: klensy Date: Sun, 14 Jan 2024 15:16:23 +0300 Subject: [PATCH 2/2] 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()) }) }