On 01/12/2014 09:40 PM, Johannes Schlüter wrote:
> On Sat, 2014-01-11 at 20:15 +0100, Stefan Neufeind wrote:
>> -static int php_do_open_temporary_file(const char *path, const char *pfx, char
>> **opened_path_p TSRMLS_DC)
>> +static int php_do_open_temporary_file(const char *path, const char *pfx, char
>> **opened_path_p TSRMLS_DC, const char *suffix)
>
> TSRM should be the last parameter, move your new one in front of it.
>
>> -PHPAPI int php_open_temporary_fd_ex(const char *dir, const char *pfx, char
>> **opened_path_p, zend_bool open_basedir_check TSRMLS_DC)
>> +PHPAPI int php_open_temporary_fd_ex(const char *dir, const char *pfx, char
>> **opened_path_p, zend_bool open_basedir_check TSRMLS_DC, const char *suffix)
>
>> -PHPAPI int php_open_temporary_fd(const char *dir, const char *pfx, char **opened_path_p
>> TSRMLS_DC)
>> +PHPAPI int php_open_temporary_fd(const char *dir, const char *pfx, char **opened_path_p
>> TSRMLS_DC, const char *suffix)
>
>> -PHPAPI FILE *php_open_temporary_file(const char *dir, const char *pfx, char
>> **opened_path_p TSRMLS_DC)
>> +PHPAPI FILE *php_open_temporary_file(const char *dir, const char *pfx, char
>> **opened_path_p TSRMLS_DC, const char *suffix)
> [...]
>
> These are API changes so it could only be in 5.6.0. Eventually it might
> be added if instead of changing APIs new functions might be introduced
> and the old ones call the new ones.
> (also TSRM should be last last there, too)
>
> Aside for that: A test would be good.
>
> For not loosing it either create a feature request at bugs.php.net or
> pull request at github.
Hi,
I've sent a reply to Ferenc and David some days ago. But maybe somebody
else could also help get this forward a bit.
RFC is here, together with a pull-request:
https://wiki.php.net/rfc/tempnam-suffix
I also tried creating a testcase. For some reason it fails - but from
the commandline I don't see why. I guess it's because my ".png"-example
fails to validate against the expected result or something?
The actual output of the script for testing is fine for me though.
Would be awesome if somebody could pick this up please.
Kind regards,
Stefan