Skip to content

Commit

Permalink
chore: Remove outdated todos
Browse files Browse the repository at this point in the history
  • Loading branch information
Nukesor committed Jun 16, 2023
1 parent 458eca3 commit cc98124
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 0 additions & 2 deletions src/utils/formatting/content_split/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -95,8 +95,6 @@ pub fn split_line(line: &str, info: &ColumnDisplayInfo, delimiter: char) -> Vec<
// Only add delimiter, if we're not on a fresh line
if !new_line {
current_line.push(delimiter);
// TODO: This has been commented by accident.
//remaining_width = remaining_width.saturating_sub(1);
}

let (mut next, mut remaining) = split_long_word(remaining_width, &next);
Expand Down
2 changes: 1 addition & 1 deletion tests/all/property_test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ fn columns_and_rows() -> impl Strategy<
cell_alignments.push(cell_alignment());
}
// Add a strategy that creates random cell content with a length of 0 to column_count
// TODO: Move this back to ".*" once we properly handle multi-space UTF-8 chars.
//
// UTF-8 characters completely break table alignment in edge-case situations (e.g. 1 space columns).
// UTF-8 characters can be multiple characters wide, which conflicts with the 1 space
// column fallback, as well as fixed-width-, percental- and max-column-constraints.
Expand Down

0 comments on commit cc98124

Please sign in to comment.