Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:17773
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Date: Wed, 10 Aug 2005 11:45:22 +0200
Reply-To: Marcus Boerger <helly@php.net>
Message-ID: <124244453.20050810114522@marcus-boerger.de>
To: Ron Korving <r.korving@xit.nl>
Cc: internals@lists.php.net
In-Reply-To: <DA.92.24081.E14A9F24@pb1.pair.com>
References: <BD.4F.04646.AA6C8F24@pb1.pair.com>
	<21.AF.04646.20AC8F24@pb1.pair.com>
	<5.1.0.14.2.20050809123954.03491eb0@localhost>
	<DA.92.24081.E14A9F24@pb1.pair.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Re: drastic memory consumption with a sequenceof exceptions
From: helly@php.net (Marcus Boerger)

Hello Ron,

Wednesday, August 10, 2005, 8:47:25 AM, you wrote:

> Okay Andi, the script in this message is as simple as it gets. I used a
> syslog file to create a load of data in this case, but of course you can use
> any (text) file for this.

> #!/usr/bin/php5
> <?
>   function process()
>   {
>     $data = file("/var/log/syslog.0");

>     foreach ($data as $line)
>       throw new Exception("error");

What makes you think the file would be closed. Erm the array deleted? And
that is your problem you are unnecessarily reading all the file into an
array. Try using the same with FileObject instead, which should reduce
memory usage a lot:

foreach(FileObject($file) as $data) throw new Exception("error");

But of course you only did that to demonstrate the memory problem which in
fact simply means that we don't do a full stack cleanup like c++ would do
here (php != c++).

In php we do the cleanup at script termination. That is also the reason
why php is not perfectly suitable for console apps especially when it comes
to daemons.

best regards
marcus