ext/intl: UConverter remove few useless zval_ptr_dtor.

[devnexen]

No description provided.

[TimWolla]

I don't agree with this one. It makes the code needlessly harder to understand / easier to introduce bugs. For php_converter_from_u_callback() specifically it also looks like zargs[2] could also avoid the dtor'ing, which is a case in point.

In fact, I would even rewrite that to:

        for (size_t i = 0; i < countof(zargs); i++) {
            zval_ptr_dtor(&zargs[i]);
        }

to make it even more robust.

[devnexen]

zargs[0] was the most straightforward to address, I do not see as harder to understand tough ? one needless call I m for it in general.