From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4695 invoked by alias); 9 Nov 2006 21:16:03 -0000 Received: (qmail 4686 invoked by uid 22791); 9 Nov 2006 21:16:01 -0000 X-Spam-Check-By: sourceware.org Received: from heller.inter.net.il (HELO heller.inter.net.il) (192.114.186.73) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 09 Nov 2006 21:15:52 +0000 Received: from HOME-C4E4A596F7 (IGLD-84-228-6-2.inter.net.il [84.228.6.2]) by heller.inter.net.il (MOS 3.7.3a-GA) with ESMTP id AYD74830 (AUTH halo1); Thu, 9 Nov 2006 23:15:48 +0200 (IST) Date: Thu, 09 Nov 2006 21:16:00 -0000 Message-Id: From: Eli Zaretskii To: Vladimir Prus CC: gdb-patches@sources.redhat.com In-reply-to: (message from Vladimir Prus on Wed, 08 Nov 2006 21:47:57 +0300) Subject: Re: Automatically use hardware watchpoints Reply-to: Eli Zaretskii References: <200611011447.41170.vladimir@codesourcery.com> X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-11/txt/msg00058.txt.bz2 > From: Vladimir Prus > Date: Wed, 08 Nov 2006 21:47:57 +0300 > > > > This patch makes insert_bp_location automatically decide what breakpoint > > type should be used if the target provides a memory map. > > And here's a version of this patch that actually includes doc changes. Thanks. > Index: doc/gdb.texinfo > =================================================================== > RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v > retrieving revision 1.366 > diff -u -r1.366 gdb.texinfo > --- doc/gdb.texinfo 27 Oct 2006 22:23:20 -0000 1.366 > +++ doc/gdb.texinfo 1 Nov 2006 11:13:05 -0000 This part is approved, but please take care of the following gotchas: > +breakpoint address is read-only or read-write. This applies to ^^ Two spaces between sentences, please. > +breakpoints set with the ``break'' command as well as to internal > +breakpoints set by commands like ``next'' and ``finish''. For > +breakpoints set with ``hbreak'', @value{GDBN} will always use hardware > +breakpoints. Please use @code{break}, @code{next} etc. for command names. > +This automatic behaviour can be controlled with the following commands:: Passive voice alarm goes off! How about rewording this as follows: You can control this automatic behaviour with the following commands: