From 41178dff909695471323b96aa81cf5c3ed727880 Mon Sep 17 00:00:00 2001 From: Maxim Zhiburt Date: Mon, 5 Dec 2022 02:47:46 +0300 Subject: [PATCH] Try to fix #7338 (#7343) Signed-off-by: Maxim Zhiburt --- crates/nu-command/src/viewers/table.rs | 4 +-- crates/nu-command/tests/commands/table.rs | 32 +++++++++++------------ 2 files changed, 18 insertions(+), 18 deletions(-) diff --git a/crates/nu-command/src/viewers/table.rs b/crates/nu-command/src/viewers/table.rs index 9f1237e712..aa4f1ce8b0 100644 --- a/crates/nu-command/src/viewers/table.rs +++ b/crates/nu-command/src/viewers/table.rs @@ -912,8 +912,6 @@ fn convert_to_table2<'a>( } for (row, item) in input.clone().into_iter().enumerate() { - let row = if with_header { row + 1 } else { row }; - if let Some(ctrlc) = &ctrlc { if ctrlc.load(Ordering::SeqCst) { return Ok(None); @@ -935,6 +933,8 @@ fn convert_to_table2<'a>( column_width = max(column_width, width); let value = NuTable::create_cell(value.0, value.1); + + let row = if with_header { row + 1 } else { row }; data[row].push(value); } diff --git a/crates/nu-command/tests/commands/table.rs b/crates/nu-command/tests/commands/table.rs index 7739170f60..ed3ca3e929 100644 --- a/crates/nu-command/tests/commands/table.rs +++ b/crates/nu-command/tests/commands/table.rs @@ -31,8 +31,8 @@ fn table_expand_0() { "╭───┬───┬───┬───────────╮\ │ # │ a │ b │ c │\ ├───┼───┼───┼───────────┤\ - │ 1 │ 1 │ 2 │ 3 │\ - │ 2 │ 4 │ 5 │ ╭───┬───╮ │\ + │ 0 │ 1 │ 2 │ 3 │\ + │ 1 │ 4 │ 5 │ ╭───┬───╮ │\ │ │ │ │ │ 0 │ 1 │ │\ │ │ │ │ │ 1 │ 2 │ │\ │ │ │ │ │ 2 │ 3 │ │\ @@ -53,8 +53,8 @@ fn table_expand_exeed_overlap_0() { "╭───┬────────────────────────┬───┬───────────╮\ │ # │ a │ b │ c │\ ├───┼────────────────────────┼───┼───────────┤\ - │ 1 │ xxxxxxxxxxxxxxxxxxxxxx │ 2 │ 3 │\ - │ 2 │ 4 │ 5 │ ╭───┬───╮ │\ + │ 0 │ xxxxxxxxxxxxxxxxxxxxxx │ 2 │ 3 │\ + │ 1 │ 4 │ 5 │ ╭───┬───╮ │\ │ │ │ │ │ 0 │ 1 │ │\ │ │ │ │ │ 1 │ 2 │ │\ │ │ │ │ │ 2 │ 3 │ │\ @@ -73,8 +73,8 @@ fn table_expand_exeed_overlap_0() { "╭──────┬───────────────────────────────────────────────────┬─────┬─────────────╮\ │ # │ a │ b │ c │\ ├──────┼───────────────────────────────────────────────────┼─────┼─────────────┤\ - │ 1 │ xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx │ 2 │ 3 │\ - │ 2 │ 4 │ 5 │ ╭───┬───╮ │\ + │ 0 │ xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx │ 2 │ 3 │\ + │ 1 │ 4 │ 5 │ ╭───┬───╮ │\ │ │ │ │ │ 0 │ 1 │ │\ │ │ │ │ │ 1 │ 2 │ │\ │ │ │ │ │ 2 │ 3 │ │\ @@ -91,8 +91,8 @@ fn table_expand_deep_0() { "╭───┬───┬───┬────────────────────────╮\ │ # │ a │ b │ c │\ ├───┼───┼───┼────────────────────────┤\ - │ 1 │ 1 │ 2 │ 3 │\ - │ 2 │ 4 │ 5 │ ╭───┬────────────────╮ │\ + │ 0 │ 1 │ 2 │ 3 │\ + │ 1 │ 4 │ 5 │ ╭───┬────────────────╮ │\ │ │ │ │ │ 0 │ 1 │ │\ │ │ │ │ │ 1 │ 2 │ │\ │ │ │ │ │ 2 │ [list 3 items] │ │\ @@ -109,8 +109,8 @@ fn table_expand_deep_1() { "╭───┬───┬───┬────────────────╮\ │ # │ a │ b │ c │\ ├───┼───┼───┼────────────────┤\ - │ 1 │ 1 │ 2 │ 3 │\ - │ 2 │ 4 │ 5 │ [list 3 items] │\ + │ 0 │ 1 │ 2 │ 3 │\ + │ 1 │ 4 │ 5 │ [list 3 items] │\ ╰───┴───┴───┴────────────────╯" ); } @@ -123,8 +123,8 @@ fn table_expand_flatten_0() { "╭───┬───┬───┬───────────────╮\ │ # │ a │ b │ c │\ ├───┼───┼───┼───────────────┤\ - │ 1 │ 1 │ 2 │ 3 │\ - │ 2 │ 4 │ 5 │ ╭───┬───────╮ │\ + │ 0 │ 1 │ 2 │ 3 │\ + │ 1 │ 4 │ 5 │ ╭───┬───────╮ │\ │ │ │ │ │ 0 │ 1 │ │\ │ │ │ │ │ 1 │ 2 │ │\ │ │ │ │ │ 2 │ 1 1 1 │ │\ @@ -143,8 +143,8 @@ fn table_expand_flatten_1() { "╭───┬───┬───┬───────────────╮\ │ # │ a │ b │ c │\ ├───┼───┼───┼───────────────┤\ - │ 1 │ 1 │ 2 │ 3 │\ - │ 2 │ 4 │ 5 │ ╭───┬───────╮ │\ + │ 0 │ 1 │ 2 │ 3 │\ + │ 1 │ 4 │ 5 │ ╭───┬───────╮ │\ │ │ │ │ │ 0 │ 1 │ │\ │ │ │ │ │ 1 │ 2 │ │\ │ │ │ │ │ 2 │ 1,1,1 │ │\ @@ -164,8 +164,8 @@ fn table_expand_flatten_and_deep_1() { "╭───┬───┬───┬────────────────────────────────╮\ │ # │ a │ b │ c │\ ├───┼───┼───┼────────────────────────────────┤\ - │ 1 │ 1 │ 2 │ 3 │\ - │ 2 │ 4 │ 5 │ ╭───┬────────────────────────╮ │\ + │ 0 │ 1 │ 2 │ 3 │\ + │ 1 │ 4 │ 5 │ ╭───┬────────────────────────╮ │\ │ │ │ │ │ 0 │ 1 │ │\ │ │ │ │ │ 1 │ 2 │ │\ │ │ │ │ │ 2 │ ╭───┬────────────────╮ │ │\