| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Fixed problem introduced in a763929 (fixing bug #61347). | Gustavo André dos Santos Lopes | 2012-03-28 | 1 | -1/+1 |
| | | | | | The test added in 8c44c85 should now pass. | ||||
| * | Added failing test for ArrayObject::offsetExists(). | Shein Alexey | 2012-03-27 | 1 | -0/+10 |
| | | |||||
| * | Fixed tests that var_dump/print_r DOM objects. | Gustavo André dos Santos Lopes | 2012-03-23 | 1 | -18/+6 |
| | | |||||
| * | Fixed bug #61482, caused by the fix to bug #61418. | Gustavo André dos Santos Lopes | 2012-03-23 | 1 | -5/+8 |
| | | | | | | | | | | | | | | | | | | | Turns out I'd forgotten to also update the destructor for the iterator returned by DirectoryIterator. The iterator for DirectoryIterator maintains the same ->current pointer throughout its existence (the DirectoryIterator itself) and returns it (the same object) everytime a value is requested from the iterator. Moving forward the iterator only changes the object. Previous code added two references to the object in get_iterator on the account of 1) the iterator memory living in its DirectoryIterator object and 2) the object being stored in iterator->current. This seems to be unnecessary. Iterators are not responsible for incrementing the refcount of the values they yield, that's up to the caller (the engine). What matters for the iterator is that the object exists as long as the iterator exists and this can be guaranteed by incremented the refcount only once. Consequently, I only add one reference in get_iterator (and reclaim it in the iterator destructor). | ||||
| * | Revert "Revert "- Fixed bug #61418 (Segmentation fault when ↵ | Gustavo André dos Santos Lopes | 2012-03-23 | 2 | -15/+55 |
| | | | | | | | DirectoryIterator's or" - causes bug #61482" This reverts commit a89c4a34ee55686ab1430a5060e1460335fc5203. | ||||
| * | Revert "- Fixed bug #61418 (Segmentation fault when DirectoryIterator's or" ↵ | Stanislav Malyshev | 2012-03-22 | 2 | -55/+15 |
| | | | | | | | - causes bug #61482 This reverts commit 714f1ff4b37c5101b3c61ea108a3d415f41e50df. | ||||
| * | Revert "Fixed reference counting" | Dmitry Stogov | 2012-03-22 | 1 | -2/+2 |
| | | | | | This reverts commit 14af1fe6923c65a91cd77eab5a8d4326f854391d. | ||||
| * | Fixed reference counting | Dmitry Stogov | 2012-03-22 | 1 | -2/+2 |
| | | |||||
| * | - Fixed bug #61418 (Segmentation fault when DirectoryIterator's or | Gustavo André dos Santos Lopes | 2012-03-18 | 2 | -15/+55 |
| | | | | | | | FilesystemIterator's iterators are requested more than once without having had its dtor callback called in between). | ||||
| * | - Fixed memory leak when calling SplFileInfo's constructor twice | Felipe Pena | 2012-03-11 | 2 | -1/+19 |
| | | |||||
| * | Fix test | Xinchen Hui | 2012-03-11 | 1 | -0/+2 |
| | | |||||
| * | Fixed bug #61347 (inconsist isset behavior of Arrayobject) | Xinchen Hui | 2012-03-11 | 2 | -39/+76 |
| | | |||||
| * | - Fixed bug #61326 (ArrayObject comparison). | Gustavo André dos Santos Lopes | 2012-03-10 | 2 | -0/+43 |
| | | |||||
| * | Fix bug #60596: Code Cleanup Removing Superfluous If Statement | Nikita Popov | 2012-03-04 | 1 | -6/+3 |
| | | | | | | Cleanup only, no function change. Patch by ircmaxell. | ||||
| * | more verbose skip reason in test files with not so obvious extension ↵ | Olivier DOUCET | 2012-02-25 | 1 | -1/+1 |
| | | | | | requirements | ||||
| * | Fix test | Etienne Kneuss | 2012-02-01 | 1 | -3/+1 |
| | | |||||
| * | These tests don't work if run as root | Rasmus Lerdorf | 2012-01-08 | 2 | -8/+14 |
| | | |||||
| * | Make sure gc is enabled | Rasmus Lerdorf | 2012-01-08 | 1 | -26/+27 |
| | | |||||
| * | - Year++ | Felipe Pena | 2012-01-01 | 22 | -22/+22 |
| | | |||||
| * | Added tests contributions on spl_autoload and stream_context_set_option() | Patrick Allaert | 2011-12-09 | 1 | -0/+18 |
| | | | | | Thank you Jean-Marc Fontaine and Alter Way | ||||
| * | fix bug #60082 (Crash in ArrayObject() when using recursive references) | Antony Dovgal | 2011-11-18 | 1 | -1/+10 |
| | | | | | | | | | | | --Tis line, and those below, will be ignored-- M trunk/ext/spl/spl_array.c M branches/PHP_5_4/ext/spl/spl_array.c M branches/PHP_5_3/ext/spl/spl_array.c M branches/PHP_5_3/NEWS | ||||
| * | - Fixed bug #60261 (phar dos null pointer) | Felipe Pena | 2011-11-11 | 1 | -1/+4 |
| | | |||||
| * | ignore this test on FreeBSD per #60186 | Ferenc Kovacs | 2011-11-01 | 1 | -0/+3 |
| | | |||||
| * | duh | Ferenc Kovacs | 2011-11-01 | 1 | -0/+2 |
| | | |||||
| * | marking ext/spl/tests/bug60082.phpt as slow test | Ferenc Kovacs | 2011-11-01 | 1 | -0/+2 |
| | | |||||
| * | Test for #60082 | Xinchen Hui | 2011-10-19 | 1 | -0/+14 |
| | | |||||
| * | - Fixed bug #60042 (spl_autoload_call may manipulate a dangling pointer) | Felipe Pena | 2011-10-12 | 1 | -0/+1 |
| | | | | | | patch by: tom at punkave dot com | ||||
| * | Fixed bug #55807 (Wrong value for splFileObject::SKIP_EMPTY) | Hannes Magnusson | 2011-09-30 | 1 | -1/+1 |
| | | |||||
| * | Test for non-string replacement, bug#54304 | Hannes Magnusson | 2011-09-12 | 1 | -0/+26 |
| | | |||||
| * | Allow replacement to be any scalar value | Hannes Magnusson | 2011-09-12 | 1 | -1/+11 |
| | | |||||
| * | Fix cleanup routines | Hannes Magnusson | 2011-09-07 | 2 | -4/+4 |
| | | |||||
| * | fix skipif description | Xinchen Hui | 2011-09-07 | 1 | -1/+1 |
| | | |||||
| * | - use something we know it exists... | Pierre Joye | 2011-09-06 | 1 | -2/+2 |
| | | |||||
| * | - skip on win, name.. name. are not valid names | Pierre Joye | 2011-09-06 | 1 | -0/+6 |
| | | |||||
| * | Add skipifs for bug#54971 (patch by datibbaw@) | Hannes Magnusson | 2011-08-30 | 1 | -0/+4 |
| | | |||||
| * | - Make usage of new PHP_FE_END macro | Felipe Pena | 2011-07-25 | 6 | -33/+33 |
| | | |||||
| * | Fixed bug #54971 (Wrong result when using iterator_to_array with use_keys on ↵ | Pierrick Charron | 2011-06-28 | 2 | -0/+47 |
| | | | | | true) | ||||
| * | fixed temp directory names in spl tests | Knut Urdalen | 2011-06-25 | 2 | -4/+4 |
| | | |||||
| * | fixed/refactored some spl tests related to owner/group retrieval | Knut Urdalen | 2011-06-25 | 4 | -71/+56 |
| | | |||||
| * | - Possible fix for bug #54609 (Certain implementation(s) of SplFixedArray ↵ | Felipe Pena | 2011-06-05 | 1 | -4/+1 |
| | | | | | cause hard crash) | ||||
| * | - Fixed bug #54970 (SplFixedArray::setSize() isn't resizing) | Felipe Pena | 2011-06-02 | 2 | -0/+40 |
| | | |||||
| * | - Make SplFileInfo::_bad_state_ex final. | Gustavo André dos Santos Lopes | 2011-04-05 | 1 | -1/+1 |
| | | |||||
| * | - Fixed bug #54384 (Dual iterators, GlobIterator, SplFileObject and | Gustavo André dos Santos Lopes | 2011-04-04 | 4 | -59/+294 |
| | | | | | | | SplTempFileObject crash when user-space classes don't call the paren constructor). | ||||
| * | Fix common typos in the source code (Reported in Bug #54065) | Pierrick Charron | 2011-04-03 | 3 | -5/+5 |
| | | | | | | | | # External libraries were excluded # Thanks eitan at eitanadler dot com for the first patch :) | ||||
| * | - Fixed bug #54292 (Wrong parameter causes crash in ↵ | Felipe Pena | 2011-03-23 | 2 | -3/+24 |
| | | | | | SplFileObject::__construct()) | ||||
| * | - Fixed bug #54281 (Crash in non-initialized RecursiveIteratorIterator) | Felipe Pena | 2011-03-22 | 2 | -0/+19 |
| | | |||||
| * | - Fixed bug #54291 (Crash iterating DirectoryIterator for dir name starting | Gustavo André dos Santos Lopes | 2011-03-19 | 2 | -1/+18 |
| | | | | | | with \0). | ||||
| * | - Fixed bug #49608 (Using CachingIterator on DirectoryIterator instance ↵ | Felipe Pena | 2011-03-08 | 1 | -1/+3 |
| | | | | | segfaults) | ||||
| * | - Fixed SplObjectStorage::offsetSet arginfo, reported in bug #54118 | Felipe Pena | 2011-03-01 | 1 | -6/+1 |
| | | |||||
| * | - Fixed memory leak in DirectoryIterator::getExtension() and ↵ | Felipe Pena | 2011-02-20 | 1 | -8/+20 |
| | | | | | SplFileInfo::getExtension() | ||||
