Return-Path: Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 56079 invoked by uid 500); 26 Apr 2002 21:20:28 -0000 Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 56065 invoked from network); 26 Apr 2002 21:20:27 -0000 Date: Fri, 26 Apr 2002 14:15:53 -0700 Subject: Re: cvs commit: httpd-2.0/server core.c Content-Type: text/plain; charset=US-ASCII; format=flowed Mime-Version: 1.0 (Apple Message framework v481) From: "Roy T. Fielding" To: dev@httpd.apache.org Content-Transfer-Encoding: 7bit In-Reply-To: <3CC87894.8F940692@apache.org> Message-Id: X-Mailer: Apple Mail (2.481) X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N I don't understand why you didn't simply reverse the test and enclose the frequent case inside the if {} block. I assume it was just to avoid indenting a large block of code, which is not sufficient justification for a goto. A goto often has unforeseen effects on high-level optimizations that can be as bad as a pipeline flush. ....Roy