From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 347B320966; Sun, 26 Mar 2017 03:37:47 +0000 (UTC) Date: Sun, 26 Mar 2017 03:37:47 +0000 From: Eric Wong To: Dylan Thacker-Smith Cc: unicorn-public@bogomips.org Subject: Re: [PATCH] Check for Socket::TCP_INFO constant before trying to get TCP_INFO Message-ID: <20170326033747.GA11690@starla> References: <20170326014903.54312-1-Dylan.Smith@shopify.com> <20170326024112.GA28554@starla> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: List-Id: Dylan Thacker-Smith wrote: > On Sat, Mar 25, 2017 at 10:41 PM, Eric Wong wrote: > > Is the last line to set "@@tcpi_inspect_ok = true" still > > necessary? It was necessary in Jeremy's version, but I think > > you can safely omit it, now. > > No it isn't needed, so you can remove it. > > I had just started my patch before seeing his, so reverted his as part > of rebasing and this line was present before his patch. Thanks, pushed as 477a2207869ff6b11a1cdee428b55604f2caa69e to "master" of git://bogomips.org/unicorn