Convert zend_parse_parameters_none() to fast ZPP in standard lib

[kocsismate]

These were the only occurrences of zend_parse_parameters_none() in the standard extension.

[carusogabriel]

Please, see #3116, and #1665 :)

[nikic]

@carusogabriel As it's not really clear when and if at all that change is going to happen, I'm okay with doing this conversion in the meantime. However, I think if we do it, we should do it for the whole codebase, as your previous PR did.

[kocsismate]

@nikic Wasn't it said that we should only use fast ZPP for the most "popular" functions/methods? What changed since then? Binary size doesn't matter that much anymore? (Because of JIT?)

Anyway, I can help/continue the work that @carusogabriel did.

[carusogabriel]

The last time that I tried to convert some ZPP calls to the new API was #3041 (comment), might help the discussion here :)

[nikic]

@kocsismate ZEND_PARSE_PARAMETERS_NONE() specifically doesn't have code-size concerns.

On the other hand, I'm not sure how much sense this makes style-wise.

[kocsismate]

@nikic Didn't know that 😮

On the other hand, I'm not sure how much sense this makes style-wise.

Then we could do the migration only for those extensions that already use fast ZPP. I think that's a good middle ground.

[nikic]

@kocsismate Sounds reasonable. As ext/standard uses FastZPP fairly consistently (right...?) I guess your version here is fine.

[kocsismate]

ext/standard uses FastZPP fairly consistently (right...?)

Yep, only 10 occurrences of zend_parse_parameters() left in ext/standard. I'll have a look at the other extensions how much they use zend_parse_parameters_none() in the coming days.