httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: svn commit: r437781 - /httpd/httpd/branches/2.2.x/modules/proxy/mod_proxy_http.c
Date Tue, 29 Aug 2006 13:17:21 GMT
> Joe Orton wrote:
> > 
> > On Mon, Aug 28, 2006 at 06:57:09PM -0000, Jim Jagielski wrote:
> > > Author: jim
> > > Date: Mon Aug 28 11:57:09 2006
> > > New Revision: 437781
> > > 
> > > URL: http://svn.apache.org/viewvc?rev=437781&view=rev
> > > Log:
> > > Merge r437768 from trunk:
> > > 
> > > Minor nit: why make the logic more complex than it needs to
> > > be ? :)
> > > 
> > > Reviewed by: jim
> > 
> > Why commit this stuff?  2.2.x is RTC.  Merging pointless changes like
> > this will screw up any future merges from earlier code on the trunk.
> > 

And when is more readable code pointless?
-- 
===========================================================================
   Jim Jagielski   [|]   jim@jaguNET.com   [|]   http://www.jaguNET.com/
	    "If you can dodge a wrench, you can dodge a ball."

Mime
View raw message