From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 76940 invoked by alias); 10 Nov 2017 14:43:56 -0000 Mailing-List: contact cygwin-apps-help@cygwin.com; run by ezmlm Precedence: bulk Sender: cygwin-apps-owner@cygwin.com List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Mail-Followup-To: cygwin-apps@cygwin.com Received: (qmail 76827 invoked by uid 89); 10 Nov 2017 14:43:56 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-25.5 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 spammy=Hx-languages-length:790 X-HELO: limerock03.mail.cornell.edu Received: from limerock03.mail.cornell.edu (HELO limerock03.mail.cornell.edu) (128.84.13.243) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 10 Nov 2017 14:43:50 +0000 X-CornellRouted: This message has been Routed already. Received: from authusersmtp.mail.cornell.edu (granite4.serverfarm.cornell.edu [10.16.197.9]) by limerock03.mail.cornell.edu (8.14.4/8.14.4_cu) with ESMTP id vAAEhmfs022129; Fri, 10 Nov 2017 09:43:48 -0500 Received: from nothing.nyroc.rr.com (mta-68-175-129-7.twcny.rr.com [68.175.129.7] (may be forged)) (authenticated bits=0) by authusersmtp.mail.cornell.edu (8.14.4/8.12.10) with ESMTP id vAAEheGT015853 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Fri, 10 Nov 2017 09:43:48 -0500 From: Ken Brown To: cygwin-apps@cygwin.com Subject: [PATCH setup 2/5] Fix off-by-one error in download retry report Date: Fri, 10 Nov 2017 14:43:00 -0000 Message-Id: <20171110144339.12616-3-kbrown@cornell.edu> In-Reply-To: <20171110144339.12616-1-kbrown@cornell.edu> References: <20171110144339.12616-1-kbrown@cornell.edu> X-PMX-Cornell-Gauge: Gauge=XXXXX X-PMX-CORNELL-AUTH-RESULTS: dkim-out=none; X-IsSubscribed: yes X-SW-Source: 2017-11/txt/msg00043.txt.bz2 'retries' was decremented after it was tested but before it was reported in the log, so the reported number was always 1 too low. --- download.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/download.cc b/download.cc index f6aa6fc..a430f7f 100644 --- a/download.cc +++ b/download.cc @@ -245,8 +245,8 @@ do_download_thread (HINSTANCE h, HWND owner) if (errors) { // In unattended mode we retry the download, but not forever. - static int retries = 4; - if (unattended_mode && retries-- <= 0) + static int retries = 5; + if (unattended_mode && --retries <= 0) { Log (LOG_PLAIN) << "download error in unattended_mode: out of retries" << endLog; Logger ().setExitMsg (IDS_INSTALL_INCOMPLETE); -- 2.15.0