Return-Path: Delivered-To: apmail-tomcat-dev-archive@www.apache.org Received: (qmail 52135 invoked from network); 26 Jan 2009 12:13:35 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 26 Jan 2009 12:13:35 -0000 Received: (qmail 71621 invoked by uid 500); 26 Jan 2009 12:13:33 -0000 Delivered-To: apmail-tomcat-dev-archive@tomcat.apache.org Received: (qmail 70935 invoked by uid 500); 26 Jan 2009 12:13:31 -0000 Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Tomcat Developers List" Delivered-To: mailing list dev@tomcat.apache.org Received: (qmail 70924 invoked by uid 99); 26 Jan 2009 12:13:31 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 26 Jan 2009 04:13:31 -0800 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of funkman@joedog.org designates 204.74.20.252 as permitted sender) Received: from [204.74.20.252] (HELO sid.armstrong.com) (204.74.20.252) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 26 Jan 2009 12:13:25 +0000 Received: from [10.38.20.119] (tafunk-lt.americas.armstrong.com [10.38.20.119]) by sid.armstrong.com (8.13.6/8.12.8) with ESMTP id n0QC7Y58010727 for ; Mon, 26 Jan 2009 07:07:34 -0500 Message-ID: <497DA8CF.1040608@joedog.org> Date: Mon, 26 Jan 2009 07:13:03 -0500 From: Tim Funk Organization: Human Being User-Agent: Thunderbird 2.0.0.19 (Windows/20081209) MIME-Version: 1.0 To: Tomcat Developers List Subject: Re: View towards refactoring changes? References: <497D69F6.90804@mustardgrain.com> In-Reply-To: <497D69F6.90804@mustardgrain.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org IIRC - there was talk of this over a year ago. In think in the context of that talk APR also had some duplication too and was "in scope" for the merge. But the dev list archives has more details and the conclusion. -Tim Kirk True wrote: > Hi all, > > There are a number of places in NioEndpoint and JIoEndpoint that share a > good portion of common code. The accessors for common properties take up > a good chunk, but there's also some boilerplate code for checking if > we're paused, running, etc. There are a lot of places where variable and > method scope could be cleaned up too. > > Obviously I can't commit this code myself, but if I submitted patches > for it, would they be looked upon favorably or ignored as noise? :) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org For additional commands, e-mail: dev-help@tomcat.apache.org