From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16628 invoked by alias); 17 Apr 2009 05:33:10 -0000 Received: (qmail 16615 invoked by uid 22791); 17 Apr 2009 05:33:09 -0000 X-SWARE-Spam-Status: No, hits=-1.5 required=5.0 tests=AWL,BAYES_00,SARE_MSGID_LONG40,SPF_PASS X-Spam-Check-By: sourceware.org Received: from ti-out-0910.google.com (HELO ti-out-0910.google.com) (209.85.142.185) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 17 Apr 2009 05:33:04 +0000 Received: by ti-out-0910.google.com with SMTP id a1so450711tib.12 for ; Thu, 16 Apr 2009 22:32:57 -0700 (PDT) MIME-Version: 1.0 Received: by 10.110.21.17 with SMTP id 17mr2415743tiu.5.1239946377796; Thu, 16 Apr 2009 22:32:57 -0700 (PDT) In-Reply-To: <20090416235944.GP7585@adacore.com> References: <20090416173918.GP7557@adacore.com> <20090416235944.GP7585@adacore.com> Date: Fri, 17 Apr 2009 05:33:00 -0000 Message-ID: Subject: Re: [RFA] print error message if (auto) disassembly failed From: Hui Zhu To: Joel Brobecker , Tom Tromey Cc: gdb-patches@sourceware.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes 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 X-SW-Source: 2009-04/txt/msg00407.txt.bz2 Hi Guys, Sorry for late. In before, when the user step into the program that don't have line message, they will lost the the help from debuger. They will not know what will happen in next except a pc address. For now, this auto will auto output the asm code. I think it will make user experience (special for new user) better. And I think Joel's patch fix the bug. So I suggest keep it default "auto". Thanks, Hui On Fri, Apr 17, 2009 at 07:59, Joel Brobecker wrote: >> The default is "auto", which means: >> >> =A0 =A0If AUTO (which is the default), or there's no line info to determ= ine >> =A0 =A0the source line of the next instruction, display disassembly of n= ext >> =A0 =A0instruction instead. =A0*/ >> >> But I agree with you that this is the wrong default. >> I think the default should be "off". > > Would anyone object to changing the default to "off"? > > -- > Joel >