From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21141 invoked by alias); 19 Nov 2010 23:45:41 -0000 Received: (qmail 21132 invoked by uid 22791); 19 Nov 2010 23:45:40 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 19 Nov 2010 23:45:35 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 423C32BACE2; Fri, 19 Nov 2010 18:45:34 -0500 (EST) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id wtKnQwvkvCQd; Fri, 19 Nov 2010 18:45:34 -0500 (EST) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id F12D72BACD8; Fri, 19 Nov 2010 18:45:33 -0500 (EST) Received: by joel.gnat.com (Postfix, from userid 1000) id 443921457E0; Fri, 19 Nov 2010 15:45:30 -0800 (PST) Date: Fri, 19 Nov 2010 23:45:00 -0000 From: Joel Brobecker To: Thiago Jung Bauermann Cc: Jan Kratochvil , Eli Zaretskii , gdb-patches@sourceware.org Subject: Re: [patch 2/2] Implement support for PowerPC BookE masked and ranged watchpoints Message-ID: <20101119234530.GL2634@adacore.com> References: <1282074110.2606.703.camel@hactar> <1287807761.10521.423.camel@hactar> <838w1p8egs.fsf@gnu.org> <1288403952.2598.58.camel@hactar> <83wrp05f2f.fsf@gnu.org> <1288648026.3377.7.camel@hactar> <83d3qo5qkj.fsf@gnu.org> <1288905125.14606.4.camel@hactar> <20101116040116.GA19243@host0.dyn.jankratochvil.net> <1290196508.5540.101.camel@hactar> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1290196508.5540.101.camel@hactar> User-Agent: Mutt/1.5.20 (2009-06-14) 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: 2010-11/txt/msg00280.txt.bz2 > So to be honest I'm the only one actually in favor of introducing > watch-range. :-) So should I drop watch-range then? One thing that I was going to propose when I thought I was going to be the only reviewer was to split the patch a little. The are several different features being implemented by this patch. If you want, we can discuss this separately from the other features... -- Joel