From 34eb9b2664955ec7eda06dea62842bfc233d71d0 Mon Sep 17 00:00:00 2001 From: Michael Stapelberg Date: Sun, 24 Jul 2011 15:00:09 +0200 Subject: [PATCH] ipc: make 'focused' a bool --- src/ipc.c | 2 +- testcases/t/16-nestedcons.t | 2 +- testcases/t/29-focus-after-close.t | 6 +++--- testcases/t/32-move-workspace.t | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/ipc.c b/src/ipc.c index ca341dc7..b2cd482c 100644 --- a/src/ipc.c +++ b/src/ipc.c @@ -201,7 +201,7 @@ void dump_node(yajl_gen gen, struct Con *con, bool inplace_restart) { y(bool, con->urgent); ystr("focused"); - y(integer, (con == focused)); + y(bool, (con == focused)); ystr("layout"); switch (con->layout) { diff --git a/testcases/t/16-nestedcons.t b/testcases/t/16-nestedcons.t index 9bcff8c6..7763ce79 100644 --- a/testcases/t/16-nestedcons.t +++ b/testcases/t/16-nestedcons.t @@ -28,7 +28,7 @@ my $expected = { percent => undef, layout => 'default', focus => ignore(), - focused => 0, + focused => JSON::XS::false, urgent => JSON::XS::false, border => 'normal', 'floating_nodes' => ignore(), diff --git a/testcases/t/29-focus-after-close.t b/testcases/t/29-focus-after-close.t index 5f5ef4af..fa9d86b5 100644 --- a/testcases/t/29-focus-after-close.t +++ b/testcases/t/29-focus-after-close.t @@ -22,10 +22,10 @@ cmd 'split v'; my ($nodes, $focus) = get_ws_content($tmp); -is($nodes->[1]->{focused}, 0, 'split container not focused'); +ok(!$nodes->[1]->{focused}, 'split container not focused'); cmd 'focus parent'; ($nodes, $focus) = get_ws_content($tmp); -is($nodes->[1]->{focused}, 1, 'split container focused after focus parent'); +ok($nodes->[1]->{focused}, 'split container focused after focus parent'); my $third = open_empty_con($i3); @@ -50,7 +50,7 @@ cmd 'kill'; sleep 0.25; ($nodes, $focus) = get_ws_content($tmp); is($nodes->[1]->{nodes}->[0]->{id}, $second, 'second container found'); -is($nodes->[1]->{nodes}->[0]->{focused}, 1, 'second container focused'); +ok($nodes->[1]->{nodes}->[0]->{focused}, 'second container focused'); ############################################################## # another case, using a slightly different layout (regression) diff --git a/testcases/t/32-move-workspace.t b/testcases/t/32-move-workspace.t index 93bd4f8f..8730cdfb 100644 --- a/testcases/t/32-move-workspace.t +++ b/testcases/t/32-move-workspace.t @@ -35,7 +35,7 @@ my ($nodes, $focus) = get_ws_content($tmp2); is($focus->[0], $second, 'same container on different ws'); ($nodes, $focus) = get_ws_content($tmp); -is($nodes->[0]->{focused}, 1, 'first container focused on first ws'); +ok($nodes->[0]->{focused}, 'first container focused on first ws'); ################################################################### # check if floating cons are moved to new workspaces properly -- 2.39.5