Skip to content

Commit

Permalink
Fixed a rather silly bug where a function no longer overrode a functi…
Browse files Browse the repository at this point in the history
…on from the parent class. Made this function explicitly override to prevent future problems
  • Loading branch information
Martijn Otto committed Feb 18, 2014
1 parent 1ae81bf commit 9c2c6bb
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion src/boolmember.h
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ class BoolMember : public MemberInfo
* @param size Size of the name
* @param flags Additional flags
*/
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, int flags) override
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, MemberModifier flags) override
{
#if PHP_VERSION_ID >= 50400
if (flags == constMember) zend_declare_class_constant_bool(entry, name, size, _value);
Expand Down
2 changes: 1 addition & 1 deletion src/doublemember.h
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ class DoubleMember : public MemberInfo
* @param size Size of the name
* @param flags Additional flags
*/
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, int flags) override
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, MemberModifier flags) override
{
#if PHP_VERSION_ID >= 50400
if (flags == constMember) zend_declare_class_constant_double(entry, name, size, _value);
Expand Down
2 changes: 1 addition & 1 deletion src/longmember.h
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ class LongMember : public MemberInfo
* @param size Size of the name
* @param flags Additional flags
*/
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, int flags) override
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, MemberModifier flags) override
{
#if PHP_VERSION_ID >= 50400
if (flags == constMember) zend_declare_class_constant_long(entry, name, size, _value);
Expand Down
4 changes: 2 additions & 2 deletions src/memberinfo.h
Original file line number Diff line number Diff line change
Expand Up @@ -67,15 +67,15 @@ class MemberInfo
* @param size Size of the name
* @param flags Additional flags
*/
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, int flags) {};
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, MemberModifier flags) {};

/**
* Fill a function entry object
* @param entry Function entry
* @param classname Name of the class
* @param flags Is this a public method?
*/
virtual void fill(struct _zend_function_entry *entry, const char *classname, int flags) {};
virtual void fill(struct _zend_function_entry *entry, const char *classname, MemberModifier flags) {};
};

/**
Expand Down
2 changes: 1 addition & 1 deletion src/methodmember.h
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ class MethodMember : public MemberInfo, public Function
* @param classname Name of the class
* @param method Is this a public entry
*/
virtual void fill(struct _zend_function_entry *entry, const char *classname, MemberModifier flags)
virtual void fill(struct _zend_function_entry *entry, const char *classname, MemberModifier flags) override
{
// call function object
Function::fill(entry, classname, flags);
Expand Down
2 changes: 1 addition & 1 deletion src/nullmember.h
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ class NullMember : public MemberInfo
* @param size Size of the name
* @param flags Additional flags
*/
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, int flags) override
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, MemberModifier flags) override
{
#if PHP_VERSION_ID >= 50400
if (flags == constMember) zend_declare_class_constant_null(entry, name, size);
Expand Down
2 changes: 1 addition & 1 deletion src/stringmember.h
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class StringMember : public MemberInfo
* @param size Size of the name
* @param flags Additional flags
*/
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, int flags) override
virtual void declare(struct _zend_class_entry *entry, const char *name, int size, MemberModifier flags) override
{
#if PHP_VERSION_ID >= 50400
if (flags == constMember) zend_declare_class_constant_stringl(entry, name, size, _value.c_str(), _value.size());
Expand Down

0 comments on commit 9c2c6bb

Please sign in to comment.