Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:22122
Return-Path: <pierre.php@gmail.com>
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Delivered-To: mailing list internals@lists.php.net
Received: (qmail 5496 invoked by uid 1010); 5 Mar 2006 15:21:24 -0000
Delivered-To: ezmlm-scan-internals@lists.php.net
Delivered-To: ezmlm-internals@lists.php.net
Received: (qmail 5481 invoked from network); 5 Mar 2006 15:21:24 -0000
Received: from unknown (HELO lists.php.net) (127.0.0.1)
  by localhost with SMTP; 5 Mar 2006 15:21:24 -0000
X-Host-Fingerprint: 217.79.190.163 r163.red.fastwebserver.de  
Received: from ([217.79.190.163:13798] helo=localhost.localdomain)
	by pb1.pair.com (ecelerity 2.0 beta r(6323M)) with SMTP
	id BF/DA-22029-3F10B044 for <internals@lists.php.net>; Sun, 05 Mar 2006 10:21:23 -0500
To: internals@lists.php.net
Date: Sun, 5 Mar 2006 16:21:28 +0100
Message-ID: <20060305162128.5b1fdb86@localhost.localdomain>
Reply-To: pierre.php@gmail.com
X-Newsreader: Sylpheed-Claws 1.9.14 (GTK+ 2.8.6; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
X-Posted-By: 217.79.190.163
Subject: Adieu register_globals
From: pierre.php@gmail.com (Pierre)

Hello,

As discussed last year, register_globals will be removed in php6. This
patch is the first step:

http://pear.php.net/~pierre/remove_register_globals.txt

The only thing I find confusing is a comment in main/php_variables.c
php_autoglobal_merge. The comment says something and the tests
something else (register_globals AND instead of register_globals OR). I
will review all the affected tests if you consider this patch ok to be
commited. Some will be updated, other simply removed.

I did not run benchmarks, but I feel like it speeds up php a bit :)

Comments and feedbacks welcome,

--Pierre