Re: Re: cvs: php4 /sapi/apache2filter php_apache.h sapi_apache2.c

From:	Andi Gutmans	Date:	Sat, 04 Jan 2003 08:33:49 +0000
Subject:	Re: Re: cvs: php4 /sapi/apache2filter php_apache.h sapi_apache2.c
References:	1 2	Groups:	php.dev
Request:	Send a blank email to php-dev+get-93108@lists.php.net to get a copy of this message

At 08:25 PM 1/3/2003 +0100, Sascha Schumann wrote:

On Fri, 3 Jan 2003, Anantha Kesari H Y wrote:

hyanantha             Fri Jan  3 10:59:02 2003 EDT

Modified files:

    /php4/sapi/apache2filter  php_apache.h sapi_apache2.c

Log: Modifications for NetWare.

    I need to say it again.

    These modifications are *extremely* ugly.

    It's like spilling a glass of wine over important documents.
    You won't be able to remove the spilled substance without
    expensive labor.

    These modifications lack proper factoring.  They repeat the
    same kind of change all over our code base.  Instead of
    hiding some of netware's properties behind appropiate
    macros and typedefs, you duplicate existing code in a
    slightly changed manner all over the place.

    What do you guys think about this?

I agree. The novell changes are butt ugly. They are just hacking at our code instead of fixing it nicely. Andi