Skip to content

Commit

Permalink
undo mut before raising err
Browse files Browse the repository at this point in the history
  • Loading branch information
nameexhaustion committed Feb 20, 2025
1 parent 40330c5 commit b815951
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions crates/polars-core/src/frame/horizontal.rs
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,11 @@ impl DataFrame {
self.columns.extend_from_slice(columns);

if cfg!(debug_assertions) {
DataFrame::validate_columns_slice(&self.columns).unwrap();
if let err @ Err(_) = DataFrame::validate_columns_slice(&self.columns) {
// Reset DataFrame state to before extend.
self.columns.truncate(self.columns.len() - columns.len());
err.unwrap();
}
}

if let Some(c) = self.columns.first() {
Expand All @@ -43,7 +47,11 @@ impl DataFrame {
self.clear_schema();
self.columns.extend_from_slice(columns);

DataFrame::validate_columns_slice(&self.columns)?;
if let err @ Err(_) = DataFrame::validate_columns_slice(&self.columns) {
// Reset DataFrame state to before extend.
self.columns.truncate(self.columns.len() - columns.len());
err?;
}

if let Some(c) = self.columns.first() {
unsafe { self.set_height(c.len()) };
Expand Down

0 comments on commit b815951

Please sign in to comment.