From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id 6FF4D3857836 for ; Mon, 7 Mar 2022 22:14:07 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 6FF4D3857836 Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-435-rkGmA3-LMDCcg4mIqXUDOw-1; Mon, 07 Mar 2022 17:14:06 -0500 X-MC-Unique: rkGmA3-LMDCcg4mIqXUDOw-1 Received: by mail-wm1-f71.google.com with SMTP id a26-20020a7bc1da000000b003857205ec7cso218412wmj.2 for ; Mon, 07 Mar 2022 14:14:05 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=2+EJ58YWboE8J/B2OT4pK6DnZ/EPy7yzlXeOyVdl+DY=; b=ml4c+YRg6eC/Y68i76pwU8G2tSdsSEQwB/Cu4nVZAG7htugpA0LURrj9PyxXKw3P5b speXHLRxufTubgmNxGk2eEyx2iV+t9obwD/RXrnv+jZ+z/OY7pkVvX3h6DqvMFLETPA7 ot4Tqyp8aQCne7aGHvYfHK85+YMQ0rERg8/RM6bd39UNuXu8wZcy135562gmhoGOiqIV Y0PGoBhmHvzWOV1B3ipJSE3xZcLqenqjkFE/FK3FoxQygE6n+l+nK2KbORdAV1szFR/h /23gVpN90YDo0pbzv1Rvg4LTP6VWSXtlAEQCgeZYBDypWfYPbMeBI0W3AQqnK5wK0E8L QJCg== X-Gm-Message-State: AOAM533BWjaWK8s3NqyNwG4HpdAtpR9Ninyi3U10RueI2vhEnxrvNKLs U8pnk7bdzqeB+9l2uZM/+GP9DTR7SipAJiRmFiHlXgPH7ElLZHUHYUEZOoO+JWoI4dCLUn1QA0j 2psEzv8TMibZCPI9G4vk2uChlkKqyhJ51tCpjW4n0wTHWJebBoAs1unbzz1gmGnT/uLXj6xGuIA == X-Received: by 2002:a1c:2744:0:b0:382:a9b7:1c8a with SMTP id n65-20020a1c2744000000b00382a9b71c8amr882268wmn.187.1646691244515; Mon, 07 Mar 2022 14:14:04 -0800 (PST) X-Google-Smtp-Source: ABdhPJyhq2VGcGBGMwpr/2I6Ygo/d4tquUGcUXmFg4zd6+U+TcAnbmXiLLhM1O3Y5pzXXz6qBVZGhA== X-Received: by 2002:a1c:2744:0:b0:382:a9b7:1c8a with SMTP id n65-20020a1c2744000000b00382a9b71c8amr882246wmn.187.1646691244200; Mon, 07 Mar 2022 14:14:04 -0800 (PST) Received: from localhost (host86-134-151-205.range86-134.btcentralplus.com. [86.134.151.205]) by smtp.gmail.com with ESMTPSA id 2-20020a1c1902000000b00380d3873d6asm464887wmz.43.2022.03.07.14.14.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Mar 2022 14:14:03 -0800 (PST) From: Andrew Burgess To: gdb-patches@sourceware.org Cc: Andrew Burgess Subject: [PATCHv3 10/15] gdb/tui: fairer distribution of excess space during apply Date: Mon, 7 Mar 2022 22:13:42 +0000 Message-Id: <12dd572021d8f0bc711dc711b64cfbcfe8364a5f.1646691034.git.aburgess@redhat.com> X-Mailer: git-send-email 2.25.4 In-Reply-To: References: MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII" X-Spam-Status: No, score=-11.6 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, KAM_SHORT, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H5, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 07 Mar 2022 22:14:09 -0000 When applying layouts gdb computes the size of each window (or rather, each sub-layout within a layout) using integer arithmetic. As this rounds down the results, then, when all sub-layouts are sized, there is the possibility that we have some space left over. Currently, this space is just assigned to an arbitrary sub-layout. This can result in some unbalanced results. Consider this set of steps with current master: (gdb) tui enable (gdb) layout regs (gdb) info win Name Lines Columns Focus regs 7 80 src 9 80 (has focus) status 1 80 cmd 8 80 Notice the weird split between the src and regs windows, the original layout specification has these windows given equal weight. The problem is that, with rounding, both the regs and src windows are initially sized to 7, the extra 2 lines are then arbitrarily added to the src window. In this commit, rather than add all the extra space to one single window, I instead hand out the extra space 1 line at a time, looping over all the sub-layouts. We take care to respect the min/max sizes, and so, we now get this result: (gdb) tui enable (gdb) layout regs (gdb) info win Name Lines Columns Focus regs 8 80 src 8 80 (has focus) status 1 80 cmd 8 80 This looks more natural to me. This is obviously a change in behaviour, and so, lots of the existing tests need to be updated to take this into account. None of the changes are huge, it's just a line or two (or column for width) moved between windows. --- gdb/testsuite/gdb.tui/basic.exp | 4 ++-- gdb/testsuite/gdb.tui/empty.exp | 14 +++++++---- gdb/testsuite/gdb.tui/regs.exp | 4 ++-- gdb/testsuite/gdb.tui/winwidth.exp | 17 ++++++------- gdb/tui/tui-layout.c | 38 ++++++++++++++++++++++++------ 5 files changed, 53 insertions(+), 24 deletions(-) diff --git a/gdb/testsuite/gdb.tui/basic.exp b/gdb/testsuite/gdb.tui/basic.exp index 0e9f2e3eab2..7afb28b0858 100644 --- a/gdb/testsuite/gdb.tui/basic.exp +++ b/gdb/testsuite/gdb.tui/basic.exp @@ -104,5 +104,5 @@ Term::command "layout split" Term::check_contents "split layout contents" \ "$main_line *$main_re.*$hex
" -Term::check_box "source box in split layout" 0 0 80 7 -Term::check_box "asm box in split layout" 0 6 80 9 +Term::check_box "source box in split layout" 0 0 80 8 +Term::check_box "asm box in split layout" 0 7 80 8 diff --git a/gdb/testsuite/gdb.tui/empty.exp b/gdb/testsuite/gdb.tui/empty.exp index ef14fe4dac0..3efa55d812e 100644 --- a/gdb/testsuite/gdb.tui/empty.exp +++ b/gdb/testsuite/gdb.tui/empty.exp @@ -35,26 +35,26 @@ if {![Term::enter_tui]} { set layouts { {src src {{0 0 80 15}} {{0 0 90 26}} {{"no source" "No Source Available"}}} - {regs src-regs {{0 0 80 7} {0 6 80 9}} {{0 0 90 13} {0 12 90 13}} + {regs src-regs {{0 0 80 8} {0 7 80 8}} {{0 0 90 13} {0 12 90 13}} { {"no source" "No Source Available"} {"no regs" "Register Values Unavailable"} }} - {asm asm {{0 0 80 13}} {{0 0 90 26}} + {asm asm {{0 0 80 15}} {{0 0 90 26}} { {"no asm" "No Assembly Available"} }} - {regs asm-regs {{0 0 80 7} {0 6 80 9}} {{0 0 90 13} {0 12 90 13}} + {regs asm-regs {{0 0 80 8} {0 7 80 8}} {{0 0 90 13} {0 12 90 13}} { {"no asm" "No Assembly Available"} {"no regs" "Register Values Unavailable"} }} - {split split {{0 0 80 6} {0 5 80 8}} {{0 0 90 13} {0 12 90 13}} + {split split {{0 0 80 8} {0 7 80 8}} {{0 0 90 13} {0 12 90 13}} { {"no source" "No Source Available"} {"no asm" "No Assembly Available"} }} - {regs split-regs {{0 0 80 6} {0 5 80 8}} {{0 0 90 13} {0 12 90 13}} + {regs split-regs {{0 0 80 8} {0 7 80 8}} {{0 0 90 13} {0 12 90 13}} { {"no asm" "No Assembly Available"} {"no regs" "Register Values Unavailable"} @@ -97,5 +97,9 @@ foreach layout $layouts { check_text $text_list } Term::resize 24 80 + with_test_prefix "80x24 again" { + check_boxes $small_boxes + check_text $text_list + } } } diff --git a/gdb/testsuite/gdb.tui/regs.exp b/gdb/testsuite/gdb.tui/regs.exp index 178eba03f19..2f3482f5d38 100644 --- a/gdb/testsuite/gdb.tui/regs.exp +++ b/gdb/testsuite/gdb.tui/regs.exp @@ -38,8 +38,8 @@ if {![Term::enter_tui]} { Term::check_contents "source at startup" "\\|.*21 *return 0" Term::command "layout regs" -Term::check_box "register box" 0 0 80 7 -Term::check_box "source box in regs layout" 0 6 80 9 +Term::check_box "register box" 0 0 80 8 +Term::check_box "source box in regs layout" 0 7 80 8 set text [Term::get_line 1] # Just check for any register window content at all. diff --git a/gdb/testsuite/gdb.tui/winwidth.exp b/gdb/testsuite/gdb.tui/winwidth.exp index b0a838b578f..1767b25c8d3 100644 --- a/gdb/testsuite/gdb.tui/winwidth.exp +++ b/gdb/testsuite/gdb.tui/winwidth.exp @@ -39,24 +39,25 @@ with_test_prefix "original window sizes" { } with_test_prefix "after src +5" { - Term::check_box "source box" 0 0 44 15 - Term::check_box "asm box" 43 0 37 15 + Term::check_box "source box" 0 0 45 15 + Term::check_box "asm box" 44 0 36 15 Term::command "winwidth asm -5" } with_test_prefix "after asm -5" { - Term::check_box "source box" 0 0 48 15 - Term::check_box "asm box" 47 0 33 15 + Term::dump_screen + Term::check_box "source box" 0 0 50 15 + Term::check_box "asm box" 49 0 31 15 Term::command "winwidth asm +8" } with_test_prefix "after asm +8" { - Term::check_box "source box" 0 0 39 15 - Term::check_box "asm box" 38 0 42 15 + Term::check_box "source box" 0 0 42 15 + Term::check_box "asm box" 41 0 39 15 Term::command "winwidth src -2" } with_test_prefix "after src -2" { - Term::check_box "source box" 0 0 36 15 - Term::check_box "asm box" 35 0 45 15 + Term::check_box "source box" 0 0 40 15 + Term::check_box "asm box" 39 0 41 15 } diff --git a/gdb/tui/tui-layout.c b/gdb/tui/tui-layout.c index 8758bfe70e7..4988d48a8d5 100644 --- a/gdb/tui/tui-layout.c +++ b/gdb/tui/tui-layout.c @@ -810,11 +810,11 @@ tui_layout_split::apply (int x_, int y_, int width_, int height_) info[i].size = info[i].max_size; if (info[i].size < info[i].min_size) info[i].size = info[i].min_size; - /* If there is any leftover size, just redistribute it to the - last resizeable window, by dropping it from the allocated - size. We could try to be fancier here perhaps, by - redistributing this size among all windows, not just the - last window. */ + /* Keep a total of all the size we've used so far (we gain some + size back if this window can share a border with a preceding + window). Any unused space will be distributed between all of + the other windows (while respecting min/max sizes) later in + this function. */ used_size += info[i].size; if (info[i].share_box) --used_size; @@ -834,9 +834,33 @@ tui_layout_split::apply (int x_, int y_, int width_, int height_) } /* Allocate any leftover size. */ - if (available_size >= used_size && last_index != -1) + if (available_size > used_size && last_index != -1) { - info[last_index].size += available_size - used_size; + /* Loop over all windows until all available space is used up. */ + bool found_window_that_can_grow_p = true; + for (int idx = last_index; + available_size > used_size; + idx = (idx + 1) % m_splits.size ()) + { + /* Once we have visited all of the windows, check that we did + manage to allocate some more space. This prevents us getting + stuck in the loop forever if we can't allocate anything + more. */ + if (idx == last_index) + { + if (!found_window_that_can_grow_p) + break; + found_window_that_can_grow_p = false; + } + + if (available_size > used_size + && info[idx].size < info[idx].max_size) + { + found_window_that_can_grow_p = true; + info[idx].size += 1; + used_size += 1; + } + } if (debug_tui) { -- 2.25.4