From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8810 invoked by alias); 10 Sep 2014 12:50:41 -0000 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 Received: (qmail 8801 invoked by uid 89); 10 Sep 2014 12:50:40 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.7 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.3.2 X-HELO: rock.gnat.com Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Wed, 10 Sep 2014 12:50:37 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 7480C11659C; Wed, 10 Sep 2014 08:50:35 -0400 (EDT) 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 JJd1iVnS08yJ; Wed, 10 Sep 2014 08:50:35 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 3E963116468; Wed, 10 Sep 2014 08:50:35 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id 66044477FF; Wed, 10 Sep 2014 05:50:37 -0700 (PDT) Date: Wed, 10 Sep 2014 12:50:00 -0000 From: Joel Brobecker To: Gabriel Krisman Bertazi Cc: gdb-patches@sourceware.org Subject: Re: [PING] [PATCH] Fix gdb.fortran/array-element.exp failures. Message-ID: <20140910125037.GN28404@adacore.com> References: <8761jbbu6l.fsf@Argo.krisman.be> <87lhrv36pm.fsf@Argo.krisman.be> <87zjf3sqrd.fsf@krisman.be> <87r4048546.fsf_-_@krisman.be> <20140909130901.GG28404@adacore.com> <87d2b4c07h.fsf@anubis.Home> <20140909154531.GJ28404@adacore.com> <87ppf4acb5.fsf@anubis.Home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87ppf4acb5.fsf@anubis.Home> User-Agent: Mutt/1.5.21 (2010-09-15) X-SW-Source: 2014-09/txt/msg00297.txt.bz2 > > Joel, thanks for your clarification. > > Ok, so we got this situation: > > The original testcase sets a breakpoint at the label continue and > resumes execution until we reach it. On the Fortran file, this means > the inferior has iterated over the whole loop before reaching the > breakpoint for the first time. Then, the original testcase issues > another continue command, causing the inferior to finish the execution > earlier than expected, since we still want to make a final test on > whether we print the second element. This causes the two test failures. > > My guess is that the original author meant to break after each loop > iteration, instead of going all the way until the continue label. > > Nevertheless, stepping over a single iteration or stopping after the > entire loop has no impact on the test results. So, what my patch does > is simply remove the second "continue" command that would prematurely > end inferior's execution, so we can actually test whether both elements > are printed correctly after executing the loop. > > Other than that, when I first submitted this patch, Sergio asked me to > simplify the testcase, because it felt unusual. That is what the other > modifications are about. OK, thank you for the explanation of the issue. > 2014-08-17 Gabriel Krisman Bertazi > > * gdb.fortran/array-element.exp: Remove wrong "continue" > command. Simplify test case. OK to push. My only comment is that simplications are indeed good, but it is better if you can submit those separately from other changes. It's easier to review the patch series that way, and it also allows us separate the real change from the enhancement which is expected to be a no-op. Thanks :) -- Joel