On 04.12.2020 at 13:35, Aimeos | Norbert Sendetzky wrote:

> Am 04.12.20 um 13:23 schrieb Christoph M. Becker:
>
>> For clarity, it might be preferable to introduce a new static creation
>> method for this.  The implementation could probably use
>> xmlReaderForIO()[1], instead of xmlReaderForFile().
>
> Wouldn't xmlReaderForFd() be more suitable or is this only for file
> handles, not for streams?

xmlReaderForFd() is only for file descriptors; PHP streams are more general.

> What would be a good name for the new method? XMLReader::stream()?

Maybe XMLReader::createForStream(), but I think the name is secondary;
someone would have to come up at least with a draft implementation first.

Regards,
Christoph

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: https://www.php.net/unsub.php

Reply via email to