Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:13266 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 79108 invoked by uid 1010); 11 Oct 2004 22:47:49 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 79056 invoked from network); 11 Oct 2004 22:47:48 -0000 Received: from unknown (HELO mail.zend.com) (80.74.107.235) by pb1.pair.com with SMTP; 11 Oct 2004 22:47:48 -0000 Received: (qmail 30696 invoked from network); 11 Oct 2004 22:47:45 -0000 Received: from localhost (HELO AndiNotebook.zend.com) (127.0.0.1) by localhost with SMTP; 11 Oct 2004 22:47:45 -0000 Message-ID: <5.1.0.14.2.20041011154514.043e1620@localhost> X-Sender: andi@localhost X-Mailer: QUALCOMM Windows Eudora Version 5.1 Date: Mon, 11 Oct 2004 15:47:39 -0700 To: "Frank M. Kromann" Cc: In-Reply-To: <10975341553680000@9866357972520000.9866341568840000> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; format=flowed Subject: Re: [PHP-DEV] [PATCH] Compiling bcompiler with PHP5 From: andi@zend.com (Andi Gutmans) References: <10975341553680000@9866357972520000.9866341568840000> It'd be smarter for bcompiler to set the correct destructor during unserializing then doing the comparisons it does during serialization. I encourage the author to fix this. There's no good reason to do it this way. Andi At 03:35 PM 10/11/2004 -0700, Frank M. Kromann wrote: >Hi Andi, > >I was just trying to compile and use the latest version of pecl/becompiler >on win32. When I compile as shared these needs to be exported. I would >prefere to have this extension as shared as it's not used that often. I >have not testet on Linux, but I would asume the same is the case there. > >Another fix could be to change the bcompiler to not use these >symbols/functions, but that would be a step backwards. > >- Frank > > > Are you sure it needs access to these functions? That seems kind of >strange... > > They aren't really supposed to be exported. If there's a real reason >I'll > > export them. > > > > Andi > > > > At 12:33 PM 10/11/2004 -0700, Frank M. Kromann wrote: > > >Hello, > > > > > >Here is a small patch to Zend taht will allow the current version of > > >pecl/bcompiler to compile with PHP and ZendEngine2. > > > > > >http://kromann.info/Zend.patch > > > > > >Would it be possible for someone with karma to commit this ? > > > > > >- Frank > > > > > >-- > > >PHP Internals - PHP Runtime Development Mailing List > > >To unsubscribe, visit: http://www.php.net/unsub.php > >