From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24917 invoked by alias); 25 Apr 2002 21:16:37 -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 24890 invoked from network); 25 Apr 2002 21:16:36 -0000 Received: from unknown (HELO cygnus.com) (205.180.83.203) by sources.redhat.com with SMTP; 25 Apr 2002 21:16:36 -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 OAA11316; Thu, 25 Apr 2002 14:16:29 -0700 (PDT) Message-ID: <3CC86F55.CA45E27B@redhat.com> Date: Thu, 25 Apr 2002 14:16:00 -0000 From: Michael Snyder Organization: Red Hat, Inc. X-Accept-Language: en MIME-Version: 1.0 To: "David S. Miller" CC: gdb-patches@sources.redhat.com Subject: Re: [RFA] Improve Sparc epilogue analysis References: <3CC73C70.A4BBA9D7@redhat.com> <20020424.170539.73357679.davem@redhat.com> <3CC74F72.6B0F8C8B@redhat.com> <20020424.180531.51276912.davem@redhat.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-SW-Source: 2002-04/txt/msg01042.txt.bz2 "David S. Miller" wrote: > > From: Michael Snyder > Date: Wed, 24 Apr 2002 17:36:02 -0700 > > The parts that are NOT mentioned in the changelog (your changes > to the logic in sparc_init_extra_frame_info) are not OK, first > because you didn't mention them in the changelog, and second > because I can't convince myself that they are right. > > Michael, I hate keeping on about this, but I did mention > them in my changes: > > (sparc_init_extra_frame_info): Use sparc_skip_prologue to find > prologue bounds instead of looking at line number info by hand. That entry says nothing about changing the logic or the functional behavior. > I know you're going to explode at me, Why would you say that? Have I exploded at you up till now? > but you seem to make a lot of errors reviewing my changes. If so, I would like to know about it. Please detail one of my errors for the group. > Please settle down, take your time. I'm going to remind you of this comment the next time you rail about my not reviewing one of your changes on a Saturday. > At > this point I'd much rather you delay than review things haphazardly. And I'd much rather you consider my reviews seriously, rather than assume they are haphazard or erroneous. I have been doing this for a while, you know... Your change to sparc_init_extra_frame_info changes its logic and its black-box behavior. You have said nothing about why you made this change. It's not my job to guess why you did it, it's my job to reject your change until you explain it. It's also not my job to take shit from you. Change your attitude, David. Change it now.