From ba002c5b9cb86626f247e22fd5c4b2de5e966748 Mon Sep 17 00:00:00 2001 From: Greg Beaver Date: Mon, 11 Feb 2008 17:13:18 +0000 Subject: [PATCH] revert last commit to phar_object.c fix open file pointers issue on windows --- ext/phar/phar.c | 7 +++++++ ext/phar/phar_object.c | 11 ----------- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/ext/phar/phar.c b/ext/phar/phar.c index d2fc3d6d4e..c255bdb3b4 100644 --- a/ext/phar/phar.c +++ b/ext/phar/phar.c @@ -213,6 +213,13 @@ int phar_archive_delref(phar_archive_data *phar TSRMLS_DC) /* {{{ */ phar_destroy_phar_data(phar TSRMLS_CC); } return 1; + } else if (!phar->refcount) { + if (phar->fp) { + /* close open file handle - allows removal or rename of + the file on windows, which has greedy locking */ + php_stream_close(phar->fp); + phar->fp = NULL; + } } return 0; } diff --git a/ext/phar/phar_object.c b/ext/phar/phar_object.c index 27c95cc452..2493ee154a 100755 --- a/ext/phar/phar_object.c +++ b/ext/phar/phar_object.c @@ -1172,16 +1172,6 @@ PHP_METHOD(Phar, getSupportedCompression) return; \ } -/* {{{ proto void Phar::__destruct() - * remove reference count of phar - */ -PHP_METHOD(Phar, __destruct) -{ - PHAR_ARCHIVE_OBJECT(); - - phar_archive_delref(phar_obj->arc.archive TSRMLS_CC); -} - static int phar_build(zend_object_iterator *iter, void *puser TSRMLS_DC) /* {{{ */ { zval **value; @@ -3449,7 +3439,6 @@ zend_function_entry php_archive_methods[] = { PHP_ME(Phar, __construct, arginfo_phar___construct, ZEND_ACC_PRIVATE) #else PHP_ME(Phar, __construct, arginfo_phar___construct, ZEND_ACC_PUBLIC) - PHP_ME(Phar, __destruct, NULL, ZEND_ACC_PUBLIC) PHP_ME(Phar, startBuffering, NULL, ZEND_ACC_PUBLIC) PHP_ME(Phar, stopBuffering, NULL, ZEND_ACC_PUBLIC) PHP_ME(Phar, compressAllFilesGZ, NULL, ZEND_ACC_PUBLIC) -- 2.50.1