shifathosn
/
gridcraft
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #643 from MarekSpodBiedry:issue-512

This commit is contained in:
Brandon Thomas 2026-10-12 23:48:30 -04:00
parent 9c4314b070 344888ceb5
commit 91138569fa
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
crates/xlsx/src/sheet_read.rs
View File

@ -541,7 +541,7 @@ pub fn parse_iso_datetime(s: &str, sys: DateSystem) -> Option<f64> {
}
pub fn parse_sqref(s: Option<&str>) -> Vec<RangeRef> {
s.unwrap_or("").split_whitespace().filter_map(RangeRef::parse).take(10_000).collect()
s.unwrap_or("").split_whitespace().filter_map(RangeRef::parse).collect()
}
fn cf_op(s: Option<&str>) -> CfOperator {
@ -974,4 +974,12 @@ mod tests {
assert_eq!(parse_iso_datetime("garbage", DateSystem::D1900), None);
assert_eq!(parse_iso_datetime("2024-13-01", DateSystem::D1900), None);
}
#[test]
fn sqref_keeps_more_than_ten_thousand_ranges() {
let sq = (1..=12_000).map(|r| format!("A{r}")).collect::<Vec<_>>().join(" ");
let ranges = parse_sqref(Some(&sq));
assert_eq!(ranges.len(), 12_000);
assert_eq!(ranges.last().copied(), RangeRef::parse("A12000"));
}
}