AFAICT, to make this change, I'd have to modify:
ZEND_API ZEND_COLD void zend_internal_type_error(zend_bool throw_exception,
const char *format, ...) /* {{{ */
To be:
ZEND_API ZEND_COLD void zend_internal_type_error(zend_bool throw_exception,
zend_class_entry *zend_error, const char *format, …) /* {{{ */
Which… would be a BC break for extensions perhaps?
There is also zend_type_error, I'm not sure how that relates, I don't think
it gets used in this case. Or, I could add
zend_(internal_?)argument_count_error and use that instead.
Thoughts?
- Davey
On Sat, Aug 6, 2016 at 4:10 AM, Thomas Punt <[email protected]> wrote:
> Hi!
>
> > From: [email protected]
> >
> > I don't like "ArgumentError", how about "WrongArgumentCountError"? Maybe
> > also "WrongArgumentsError".
>
> I'd favour simply ArgumentCountError. No need to prepend a "wrong" to the
> error class name - we can already guess it's wrong because it has resulted
> in
> an error :)
>
> -Tom
>
