From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22020 invoked by alias); 22 Aug 2002 21:19:07 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 22013 invoked from network); 22 Aug 2002 21:19:06 -0000 Received: from unknown (HELO cygnus.com) (205.180.83.203) by sources.redhat.com with SMTP; 22 Aug 2002 21:19:06 -0000 Received: from redhat.com (reddwarf.sfbay.redhat.com [172.16.24.50]) by runyon.cygnus.com (8.8.7-cygnus/8.8.7) with ESMTP id OAA06665; Thu, 22 Aug 2002 14:13:14 -0700 (PDT) Message-ID: <3D655542.AF3065A0@redhat.com> Date: Thu, 22 Aug 2002 14:24:00 -0000 From: Michael Snyder Organization: Red Hat, Inc. X-Accept-Language: en MIME-Version: 1.0 To: Grace Sainsbury CC: gdb-patches@sources.redhat.com Subject: Re: breakpoint error messages References: <20020821142627.A10117@tomago.toronto.redhat.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-SW-Source: 2002-08/txt/msg00715.txt.bz2 Grace Sainsbury wrote: > > I changed insert_breakpoints to collect all the warning messages from > failed inserts and print them with an error after trying the whole > queue. This changes the functionality slightly -- the old code stopped > after the first failed insert of a breakpoint. I also changed the > error messages to be more explicit about hardware breakpoints. > > ok? Grace, thanks for the contribution. A few implementation details. First off, you've lost some output. 1) The word "Warning: ", which is generated by the 'warning' function (which you've replaced by fprintf_unfiltered). One instance would probably be enough. 2) The output of the 'memory_error' function. 3) The msg "The same program may be running in another process" 4) The output of print_sys_errmsg, formerly called by infrun. Do you think you could work those back in? Then, just some textual edits. In one comment you say "If there wat an error", where you probably meant to say 'was'. And there's this: if (hw_breakpoint_error) fprintf_unfiltered (tmp_error_stream, "Could not insert breakpoints: ..." Seems like you might as well say "hardware breakpoints" there. Regards, Michael > > grace > > 2002-08-21 Grace Sainsbury > > * infrun.c (normal_stop, proceed): Remove call to print_sys_errmsg > when breakpoints fail. Move general breakpoint error messages to > insert_breakpoints. > * breakpoint.c (insert_breakpoints): Change warnings when > breakpoints are nto inserted to specify the type. Remove call to > memory_error when breakpoints can't be inserted. Remove multiple > calls to warning so all messages are sent to the user at once. > (delete_breakpoints): Make insert error messsages more explicit. > > ------------------------------------------------------------------------ > > patchName: patch > Type: Plain Text (text/plain)