From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from acme.spoerlein.net (acme.spoerlein.net [88.198.49.12]) by krisdoz.my.domain (8.14.5/8.14.5) with ESMTP id r4IJLVr2020832 for ; Sat, 18 May 2013 15:21:34 -0400 (EDT) Received: from localhost (acme.spoerlein.net [IPv6:2a01:4f8:131:23c2::1]) by acme.spoerlein.net (8.14.7/8.14.6) with ESMTP id r4IJLTDw033231 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sat, 18 May 2013 21:21:29 +0200 (CEST) (envelope-from uqs@spoerlein.net) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=spoerlein.net; s=dkim200908; t=1368904890; bh=JRL2vUyP3NUTjATvke9CofKv2xlmnBwPz0Ez4tsVVM4=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=bAynPSn47WaZv2jhb0wT84+em5SKZUnDyr7zGxYbP6g40pRGbnEot47+D6PuLYWWe 7rBHCNN0Sdc9+XMb9zkiuEcoF0FecJzkr1/h3mv3tg50GSEQKgODf1LnzVCRzlib6w b79ZqE5QqKNSqPIjuVjI/r+lMqVw2UdvUpODsA78= Date: Sat, 18 May 2013 21:21:29 +0200 From: Ulrich =?utf-8?B?U3DDtnJsZWlu?= To: Ingo Schwarze Cc: tech@mdocml.bsd.lv Subject: Re: [patch] remove dead code Message-ID: <20130518192129.GG2055@acme.spoerlein.net> References: <20130517010050.GY2055@acme.spoerlein.net> <20130518171207.GA13985@iris.usta.de> X-Mailinglist: mdocml-tech Reply-To: tech@mdocml.bsd.lv MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20130518171207.GA13985@iris.usta.de> User-Agent: Mutt/1.5.21 (2010-09-15) On Sat, 2013-05-18 at 19:12:07 +0200, Ingo Schwarze wrote: > Hi Ulrich, > > Ulrich Spörlein wrote on Fri, May 17, 2013 at 03:00:50AM +0200: > > > please see the following patch which removes a bit of dead code. > > With sz being invariant 0, the loop will never do anything. > > Right, committed to both bsd.lv and openbsd.org. > > > This is Coverity Scan CID 975717 > > Strangly enough, it seems that Coverity missed that **params is > unused as well. Anyway, i removed **params, too. It only reports dead "code", not unused vars :) Cheers, Uli -- To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv