From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 101404 invoked by alias); 22 May 2015 00:29:22 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 101394 invoked by uid 89); 22 May 2015 00:29:21 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY,SPF_HELO_PASS,T_RP_MATCHES_RCVD autolearn=no version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Fri, 22 May 2015 00:29:20 +0000 Received: from int-mx13.intmail.prod.int.phx2.redhat.com (int-mx13.intmail.prod.int.phx2.redhat.com [10.5.11.26]) by mx1.redhat.com (Postfix) with ESMTPS id C9BCB91E8A; Fri, 22 May 2015 00:29:19 +0000 (UTC) Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx13.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t4M0THGH018934; Thu, 21 May 2015 20:29:18 -0400 Message-ID: <555E785D.60703@redhat.com> Date: Fri, 22 May 2015 00:29:00 -0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Andrew Burgess , gdb-patches@sourceware.org Subject: Re: [PATCH] gdb: Add new 'layout cli' command. References: In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-SW-Source: 2015-05/txt/msg00590.txt.bz2 On 05/21/2015 11:06 PM, Andrew Burgess wrote: > It always seemed a little strange to me that it's possible to enter > TUI mode by typing commands at the CLI, but to leave TUI mode you need > to know the readline bindings. Agreed. > > This commit adds a new 'layout cli' command to leave tui mode, which > is in addition to the readline bindings. IIUC, "layout prev/next" doesn't cycle this layout. That seems odd. I think it either should, or this should be a separate command. A "layout cli" layout could reasonably be a layout that (with tui enabled) only showed the console window, with status line at the top, similar to "layout src; winheight src 3" (3 just because one can't go lower than 3). I supposed we could leave room for adding that later by calling this one e.g., "layout off". I notice that we have no blessed way to go back to the same layout that was current when the tui was last active. "layout current" would be odd, given that the current is "cli"... Hmm, "layout last" (like "cd -")? Not sure... A separate command ("tui enable/disable"? "set tui enabled on/off"?) would be a more direct mapping to the readline bindings. What do you think? Thanks, Pedro Alves