ext/gd using fast ZPP.

[devnexen]

No description provided.

[Girgias]

One issue but looks good to me otherwise

[thg2k]

Is there a reaon why we are doing this? I thought the idea was to use the fast ZPP only on critical functions that are invoked a lot due to the lesser readability of the code.

In this case, i can only think of a few functions where this might make sense, imagecolor[allocate|resolve] and images[x|y], for the rest, any speed gain is negligible compared to the rest of the processing time.

Or is this going to happen on the full php codebase?

[devnexen]

It is still a gain :)
It is more convenient also when there is a certain amount of arguments, it is easier to follow I find.
I m not sure it will happen to the full codebase though but can t speak for it.

[Girgias]

The main reason against it was because of code size as the "old" zpp is a function call, but the actual impact of it is not that big so it seems fine to "just" convert them.