caisin/typst
1
0
Fork 0
mirror of https://github.com/typst/typst synced 2025-05-14 04:56:26 +08:00
Code Issues Packages Projects Releases Wiki Activity

Prevent double allocation due to Content::sequence (#5084)

Browse Source
This commit is contained in:
Laurenz 2024-10-01 12:47:59 +02:00 committed by GitHub
parent 63e6150ca1
commit d03af848eb
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 8 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
crates/typst/src/foundations/content.rs
View File

@ -245,16 +245,14 @@ impl Content {
/// Create a new sequence element from multiples elements.
pub fn sequence(iter: impl IntoIterator<Item = Self>) -> Self {
let mut iter = iter.into_iter();
let Some(first) = iter.next() else { return Self::empty() };
let Some(second) = iter.next() else { return first };
SequenceElem::new(
std::iter::once(first)
.chain(std::iter::once(second))
.chain(iter)
.collect(),
)
.into()
let vec: Vec<_> = iter.into_iter().collect();
if vec.is_empty() {
Self::empty()
} else if vec.len() == 1 {
vec.into_iter().next().unwrap()
} else {
SequenceElem::new(vec).into()
}
}
/// Whether the contained element is of type `T`.