From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23842 invoked by alias); 29 May 2008 14:40:48 -0000 Received: (qmail 23828 invoked by uid 22791); 29 May 2008 14:40:44 -0000 X-Spam-Check-By: sourceware.org Received: from main.gmane.org (HELO ciao.gmane.org) (80.91.229.2) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 29 May 2008 14:40:17 +0000 Received: from list by ciao.gmane.org with local (Exim 4.43) id 1K1jIU-0005jE-6W for gdb-patches@sources.redhat.com; Thu, 29 May 2008 14:40:06 +0000 Received: from mobius.qnx.com ([209.226.137.108]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Thu, 29 May 2008 14:40:06 +0000 Received: from aristovski by mobius.qnx.com with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Thu, 29 May 2008 14:40:06 +0000 To: gdb-patches@sources.redhat.com From: Aleksandar Ristovski Subject: Re: [patch/MI] Implementation for break-catch command Date: Thu, 29 May 2008 20:46:00 -0000 Message-ID: <483EC03B.4070608@qnx.com> References: <483D9B0C.20606@qnx.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------030009040802000609010206" Cc: Vladimir Prus User-Agent: Thunderbird 2.0.0.14 (Windows/20080421) In-Reply-To: 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: 2008-05/txt/msg00755.txt.bz2 This is a multi-part message in MIME format. --------------030009040802000609010206 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-length: 1209 Eli Zaretskii wrote: >> +Where @var{event} can be one of: > > This "where" should not be capitalized, since it does not begin a new > sentence. Fixed. >> +@end smallexample > > The lines here are too long: anything longer than 66 characters runs a > risk of overflowing the page margin in the printed manual. So please > break them into shorter lines. Fixed. > >> +where @var{number} is the @value{GDBN} number for this breakpoint, > > "breakpoint" or "catchpoint"? Catchpoint, thanks. >> +greater for -break-info or -break-list which use the same output). > > Please give commands you mention here the @code markup. Done. Also cleaned up the new function (Vladimir?). 2008-05-29 Aleksandar Ristovski * breakpoint.c (catch_command_1): Remove static declaration. (print_mention_exception_catchpoint): Output information only when non-MI protocol is active. * breakpoint.h (catch_command_1): New declaration. * doc/gdb.texinfo (break-catch): Remove @ignore and add text for break-catch command. * mi/mi-cmd-break.c (mi_cmd_break_catch): New function. * mi/mi-cmds.c (mi_cmds): Add function for break-catch. * mi/mi-cmds.h (mi_cmd_break_catch): New declaration. --------------030009040802000609010206 Content-Type: text/plain; name="break-catch-200805291030.diff" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="break-catch-200805291030.diff" Content-length: 7934 /* MI Command Set - breakpoint and watchpoint commands. Copyright (C) 2000, 2001, 2002, 2007, 2008 Free Software Foundation, Inc. Contributed by Cygnus Solutions (a Red Hat company). This file is part of GDB. This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation; either version 3 of the License, or (at your option) any later version. This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details. You should have received a copy of the GNU General Public License along with this program. If not, see . */ #include "defs.h" #include "mi-cmds.h" #include "ui-out.h" #include "mi-out.h" #include "breakpoint.h" #include "gdb_string.h" #include "mi-getopt.h" #include "gdb-events.h" #include "gdb.h" #include "exceptions.h" enum { FROM_TTY = 0 }; /* Output a single breakpoint. */ static void breakpoint_notify (int b) { gdb_breakpoint_query (uiout, b, NULL); } struct gdb_events breakpoint_hooks = { breakpoint_notify, breakpoint_notify, breakpoint_notify, }; enum bp_type { REG_BP, HW_BP, REGEXP_BP }; /* Implements the -break-insert command. See the MI manual for the list of possible options. */ enum mi_cmd_result mi_cmd_break_insert (char *command, char **argv, int argc) { char *address = NULL; enum bp_type type = REG_BP; int temp_p = 0; int thread = -1; int ignore_count = 0; char *condition = NULL; int pending = 0; struct gdb_exception e; struct gdb_events *old_hooks; enum opt { HARDWARE_OPT, TEMP_OPT /*, REGEXP_OPT */ , CONDITION_OPT, IGNORE_COUNT_OPT, THREAD_OPT, PENDING_OPT }; static struct mi_opt opts[] = { {"h", HARDWARE_OPT, 0}, {"t", TEMP_OPT, 0}, {"c", CONDITION_OPT, 1}, {"i", IGNORE_COUNT_OPT, 1}, {"p", THREAD_OPT, 1}, {"f", PENDING_OPT, 0}, { 0, 0, 0 } }; /* Parse arguments. It could be -r or -h or -t, or ``--'' to denote the end of the option list. */ int optind = 0; char *optarg; while (1) { int opt = mi_getopt ("mi_cmd_break_insert", argc, argv, opts, &optind, &optarg); if (opt < 0) break; switch ((enum opt) opt) { case TEMP_OPT: temp_p = 1; break; case HARDWARE_OPT: type = HW_BP; break; #if 0 case REGEXP_OPT: type = REGEXP_BP; break; #endif case CONDITION_OPT: condition = optarg; break; case IGNORE_COUNT_OPT: ignore_count = atol (optarg); break; case THREAD_OPT: thread = atol (optarg); break; case PENDING_OPT: pending = 1; break; } } if (optind >= argc) error (_("mi_cmd_break_insert: Missing ")); if (optind < argc - 1) error (_("mi_cmd_break_insert: Garbage following ")); address = argv[optind]; /* Now we have what we need, let's insert the breakpoint! */ old_hooks = deprecated_set_gdb_event_hooks (&breakpoint_hooks); /* Make sure we restore hooks even if exception is thrown. */ TRY_CATCH (e, RETURN_MASK_ALL) { switch (type) { case REG_BP: set_breakpoint (address, condition, 0 /*hardwareflag */ , temp_p, thread, ignore_count, pending); break; case HW_BP: set_breakpoint (address, condition, 1 /*hardwareflag */ , temp_p, thread, ignore_count, pending); break; #if 0 case REGEXP_BP: if (temp_p) error (_("mi_cmd_break_insert: Unsupported tempoary regexp breakpoint")); else rbreak_command_wrapper (address, FROM_TTY); return MI_CMD_DONE; break; #endif default: internal_error (__FILE__, __LINE__, _("mi_cmd_break_insert: Bad switch.")); } } deprecated_set_gdb_event_hooks (old_hooks); if (e.reason < 0) throw_exception (e); return MI_CMD_DONE; } enum wp_type { REG_WP, READ_WP, ACCESS_WP }; /* Insert a watchpoint. The type of watchpoint is specified by the first argument: -break-watch --> insert a regular wp. -break-watch -r --> insert a read watchpoint. -break-watch -a --> insert an access wp. */ enum mi_cmd_result mi_cmd_break_watch (char *command, char **argv, int argc) { char *expr = NULL; enum wp_type type = REG_WP; enum opt { READ_OPT, ACCESS_OPT }; static struct mi_opt opts[] = { {"r", READ_OPT, 0}, {"a", ACCESS_OPT, 0}, { 0, 0, 0 } }; /* Parse arguments. */ int optind = 0; char *optarg; while (1) { int opt = mi_getopt ("mi_cmd_break_watch", argc, argv, opts, &optind, &optarg); if (opt < 0) break; switch ((enum opt) opt) { case READ_OPT: type = READ_WP; break; case ACCESS_OPT: type = ACCESS_WP; break; } } if (optind >= argc) error (_("mi_cmd_break_watch: Missing ")); if (optind < argc - 1) error (_("mi_cmd_break_watch: Garbage following ")); expr = argv[optind]; /* Now we have what we need, let's insert the watchpoint! */ switch (type) { case REG_WP: watch_command_wrapper (expr, FROM_TTY); break; case READ_WP: rwatch_command_wrapper (expr, FROM_TTY); break; case ACCESS_WP: awatch_command_wrapper (expr, FROM_TTY); break; default: error (_("mi_cmd_break_watch: Unknown watchpoint type.")); } return MI_CMD_DONE; } /* Implements the -break-catch command. See the MI manual for the list of possible options. */ enum mi_cmd_result mi_cmd_break_catch (char *command, char **argv, int argc) { char *event_name = NULL; enum bp_type type = REG_BP; int temp_p = 0; int thread = -1; int ignore_count = 0; char *condition = NULL; struct gdb_exception e; struct gdb_events *old_hooks; char argument[250]; enum opt { HARDWARE_OPT, TEMP_OPT, CONDITION_OPT, IGNORE_COUNT_OPT, THREAD_OPT }; static struct mi_opt opts[] = { {"h", HARDWARE_OPT, 0}, {"t", TEMP_OPT, 0}, {"c", CONDITION_OPT, 1}, {"i", IGNORE_COUNT_OPT, 1}, {"p", THREAD_OPT, 1}, { 0, 0, 0 } }; /* Parse arguments. It could be -r or -h or -t, or ``--'' to denote the end of the option list. */ int optind = 0; char *optarg; while (1) { int opt = mi_getopt ("mi_cmd_break_catch", argc, argv, opts, &optind, &optarg); if (opt < 0) break; switch ((enum opt) opt) { case TEMP_OPT: temp_p = 1; break; case HARDWARE_OPT: warning (_("Hardware flag ignored for catchpoints")); break; case CONDITION_OPT: condition = optarg; break; case IGNORE_COUNT_OPT: ignore_count = atol (optarg); warning (_("Ignore count not yet implemented for catchpoints")); break; case THREAD_OPT: thread = atol (optarg); warning (_("Thread option not yet implemented for catchpoints")); break; } } if (optind >= argc) error (_("mi_cmd_break_catch: Missing ")); if (optind < argc - 1) error (_("mi_cmd_break_catch: Garbage following ")); event_name = argv[optind]; if (condition != NULL) snprintf (argument, sizeof (argument), "%s %s", event_name, condition); else strcpy (argument, event_name); /* Now we have what we need, let's insert the breakpoint! */ old_hooks = deprecated_set_gdb_event_hooks (&breakpoint_hooks); /* Make sure we restore hooks even if exception is thrown. */ TRY_CATCH (e, RETURN_MASK_ALL) { switch (type) { case REG_BP: catch_command_1 (argument, temp_p, 0); break; default: internal_error (__FILE__, __LINE__, _("mi_cmd_break_catch: Bad switch.")); } } deprecated_set_gdb_event_hooks (old_hooks); if (e.reason < 0) throw_exception (e); return MI_CMD_DONE; } --------------030009040802000609010206--