From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22177 invoked by alias); 1 Jul 2009 19:55:46 -0000 Received: (qmail 22166 invoked by uid 22791); 1 Jul 2009 19:55:45 -0000 X-SWARE-Spam-Status: No, hits=-1.5 required=5.0 tests=AWL,BAYES_00,J_CHICKENPOX_23,SARE_MSGID_LONG40,SPF_PASS X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.33.17) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 01 Jul 2009 19:55:40 +0000 Received: from zps35.corp.google.com (zps35.corp.google.com [172.25.146.35]) by smtp-out.google.com with ESMTP id n61JtaET023886 for ; Wed, 1 Jul 2009 20:55:36 +0100 Received: from yxe29 (yxe29.prod.google.com [10.190.2.29]) by zps35.corp.google.com with ESMTP id n61JtXvl001922 for ; Wed, 1 Jul 2009 12:55:33 -0700 Received: by yxe29 with SMTP id 29so1581041yxe.32 for ; Wed, 01 Jul 2009 12:55:33 -0700 (PDT) MIME-Version: 1.0 Received: by 10.90.93.13 with SMTP id q13mr4564978agb.114.1246478133053; Wed, 01 Jul 2009 12:55:33 -0700 (PDT) In-Reply-To: <200907011902.19110.pedro@codesourcery.com> References: <20090523232414.BED2B846C2@localhost> <83zlbo2t2p.fsf@gnu.org> <200907011902.19110.pedro@codesourcery.com> Date: Wed, 01 Jul 2009 19:55:00 -0000 Message-ID: Subject: Re: [RFA] Fix too many "no debugging symbols found" warnings. From: Doug Evans To: Pedro Alves Cc: Eli Zaretskii , tromey@redhat.com, gdb-patches@sourceware.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-System-Of-Record: true 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-07/txt/msg00042.txt.bz2 On Wed, Jul 1, 2009 at 11:02 AM, Pedro Alves wrote: > On Wednesday 01 July 2009 18:52:50, Doug Evans wrote: >> Does anyone have any other concerns with the patch? > > Does this mean that there will be no way anymore to say > that you don't want symbol loading info? =A0I was assuming > we'd still have that option. =A0:-/ > > I find "set print symbol-loading-warnings" a bit > misnamed, since this isn't about *all* warnings. =A0In fact, > it's about *no* symbols. =A0I don't really want to get > bishedy, but, why was it that keeping the existing > option and adding a new one wasn't good? > > set print symbol-loading on|off > set print no-symbols-found-warnings on|off First, there's a bit of history here. I added "print symbol-loading" to be able to turn off the "no debugging symbols found" verbosity, but it wasn't working out the way I wanted - it was all or nothing when what I really wanted was a way to turn off the verbosity for automatically loaded libraries (for which there can be 100's or even >1000) and not, for example, executables, or anything manually done from the command line (one can think of executables as falling in this category). Secondly, it's not clear to me that users would ever want to turn off messages for things they do from the command line. i.e. "if (from_tty)" is a sufficient check for these particular messages and that's what 6.8 does (well, it also tests info_verbose). I realize scripts might cause an excess of verbosity from command-line related symbol loading, but I don't know of any existing examples of that, so I didn't want to keep two options if just for that. Thirdly, symbol-loading-warnings is a compatible renaming of symbol-loading - any distributions that have already shipped "print symbol-loading" will see a minor change in behavior in 7.0 but nothing will break. [I realize this isn't a strong argument to some. I don't have a strong opinion on the option name choice, I just mention it as it did factor into the choice.] Fourth, gdb doesn't print symbol loading messages if (!from_tty && !info_verbose). Except for the "no debugging symbols found" warning, "print symbol-loading" doesn't apply unless either of those flags are true (in current cvs). Fifth, in the past I've observed pushback on unnecessary options, and so here I was trying to "go with the flow". Put this all together and that's why the patch is the way it is. Ultimately, I don't have a strong opinion on having both options. I do have a strong opinion on providing an option to the user to let them turn off the "no debugging symbols found" message (and other such messages) from automatically loaded libraries. Thoughts?