Fix memory leak introduced by fixing bug #78221

We have to free the retrieved text content; to keep the code readable,
we extract a helper function to check for empty nodes. Unfortunate

Fix memory leak introduced by fixing bug #78221

We have to free the retrieved text content; to keep the code readable,
we extract a helper function to check for empty nodes. Unfortunately,
we cannot use xmlIsBlankNode(), because that also recognizes whitespace
only text content.

We also make sure to properly handle NULL returns from
xmlNodeGetContent().

show more ...

Fix #78221: DOMNode::normalize() doesn't remove empty text nodes

If a text node is not followed by another text node, we remove it, if
its textContent is empty.

Remove unused Git attributes ident

The $Id$ keywords were used in Subversion where they can be substituted
with filename, last revision number change, last changed date, and last
use

Remove unused Git attributes ident

The $Id$ keywords were used in Subversion where they can be substituted
with filename, last revision number change, last changed date, and last
user who changed it.

In Git this functionality is different and can be done with Git attribute
ident. These need to be defined manually for each file in the
.gitattributes file and are afterwards replaced with 40-character
hexadecimal blob object name which is based only on the particular file
contents.

This patch simplifies handling of $Id$ keywords by removing them since
they are not used anymore.

show more ...

Use zval_ptr_dtor() imstead of zval_dtor()

Export standard object handlers, to avoid indirect access

Use zend_string_release_ex() instread of zend_string_release() in places, where we sure about string persistence.

Keep initialized object_handlers structures in read-only memory.

Remove duplicated assignment

year++

year++

year++

Make sure properties are initialized before cloning

Now that they are not memset, they need to be explicitly intialized,
as zend_objects_clone_members() destroys the old property values

Make sure properties are initialized before cloning

Now that they are not memset, they need to be explicitly intialized,
as zend_objects_clone_members() destroys the old property values
first.

show more ...

Add zend_object_alloc() API

Using ecalloc() to create objects is expensive, because the
dynamic-size memset() is unreasonably slow. Make sure we only
zero the main object structure w

Add zend_object_alloc() API

Using ecalloc() to create objects is expensive, because the
dynamic-size memset() is unreasonably slow. Make sure we only
zero the main object structure with known size, as the properties
are intialized separately anyway.

Technically we do not need to zero the embedded zend_object
structure either, but as long as the memset argument is constant,
a couple more bytes don't really matter.

show more ...

Use interned strings for "magic" property of internal classes. (not copyied into SHM)

Encapsulate reference-counting primitives.
Prohibit direct update of GC_REFCOUNT(), GC_SET_REFCOUNT(), GC_ADDREF() and GC_DELREF() shoukf be instead.
Added mactros to validate reference-count

Encapsulate reference-counting primitives.
Prohibit direct update of GC_REFCOUNT(), GC_SET_REFCOUNT(), GC_ADDREF() and GC_DELREF() shoukf be instead.
Added mactros to validate reference-counting (disabled for now).
These macros are going to be used to eliminate race-condintions during reference-counting on data shared between threads.

show more ...

Refactored array creation API. array_init() and array_init_size() are converted into macros calling zend_new_array(). They are not functions anymore and don't return any values.

Implement Countable for DomNodeList and DOMNamedNodeMap (Request #74837)

Update copyright headers to 2017

Update copyright headers to 2017

Update copyright headers to 2017

Fixed #67474 (getElementsByTagNameNS and default namespace)

This bug was caused by the fact that dom_get_elements_by_tag_name_ns_raw
uses an empty string to filter on the default namespa

Fixed #67474 (getElementsByTagNameNS and default namespace)

This bug was caused by the fact that dom_get_elements_by_tag_name_ns_raw
uses an empty string to filter on the default namespace (as NULL means
'no filter'), whereas in the node itself the default namespace is
signalled by nodep->ns being null.

show more ...

Fix dom class can't be inherited by the internal class

Revert "Merge branch 'PHP-7.1'"

This reverts commit 7eac2fdf53cda5f3bd34ae64eecbb8b21a6f484b, reversing
changes made to 15b80f105cdb9a9210edd9dff7109f0951c843f4.

Revert "Fix dom class can't be inherited by the internal class"

This reverts commit 705f60e481b891d009ec9bb4c88390545ff95880.

Fix dom class can't be inherited by the internal class