Fix PDOStatement::getIterator() not throwing when parameters are provided

[alexandre-daubois]

PDOStatement::getIterator() silently fails if arguments are provided, which is inconsistent with the rest of the file.

[ndossche]

No, that macro will call zend_wrong_parameters_none_error so there is no bug here.

[alexandre-daubois]

Hmmm, then I'm not sure to get why most existing use of this macro are followed by RETURN_THROWS()?

[ndossche]

Hmmm, then I'm not sure to get why most existing use of this macro are followed by RETURN_THROWS()?

It's pointless, just convention. The only thing that macro does is assert an exception was thrown and then return.

[alexandre-daubois]

Got it, thanks 👍