From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17798 invoked by alias); 7 Jun 2009 19:28:22 -0000 Received: (qmail 17787 invoked by uid 22791); 7 Jun 2009 19:28:21 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.4) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 07 Jun 2009 19:28:16 +0000 Received: (qmail 13488 invoked from network); 7 Jun 2009 19:28:13 -0000 Received: from unknown (HELO orlando.local) (pedro@127.0.0.2) by mail.codesourcery.com with ESMTPA; 7 Jun 2009 19:28:13 -0000 From: Pedro Alves To: danny.backx@scarlet.be Subject: Re: Patch : gdbserver get_image_name on CE Date: Sun, 07 Jun 2009 19:28:00 -0000 User-Agent: KMail/1.9.10 Cc: gdb-patches@sourceware.org References: <1244366297.11918.210.camel@pavilion> <200906071916.03048.pedro@codesourcery.com> <1244401967.11918.226.camel@pavilion> In-Reply-To: <1244401967.11918.226.camel@pavilion> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200906072029.14094.pedro@codesourcery.com> 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-06/txt/msg00161.txt.bz2 On Sunday 07 June 2009 20:12:47, Danny Backx wrote: > On Sun, 2009-06-07 at 19:16 +0100, Pedro Alves wrote: > > It's fine to send unfinished patches for comments, if you'd like, but > > please say so explicitly, so I don't waste time trying to understand > > what they're for, for nothing. > > This one isn't for nothing, it fixes a problem. Sorry, you missed my point. "for nothing" refered to my waste of time, not to the usefulness of the patch. Please do realize that if you don't explain what the patch does, than whoever reviews it has to stare at the code and try to make some sense of it. If the patch isn't obvious (this one isn't obvious to me), and doesn't come with explanations, it is not going to be applied. The onus is on you to make it as easy as possible for a reviewer/maintainer to accept a patch. I really do not mean to sound harsh. It is just that it is easy to not realise that it does take time to review a patch, and the backlog isn't getting shorter... > It's just not all of the > work to get gdb/gdbserver to work for x86/ce . I understand. On Sunday 07 June 2009 20:12:47, Danny Backx wrote: > CE -> unicode, both ARM and x86. > The DLL name was in the right place, it's just the code to read it > didn't work right. It's the explanation of what's wrong with it that's missing. -- Pedro Alves