Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:123439
Message-ID: <50273236-a200-4545-8c2f-f790323f976b@gmail.com>
Date: Mon, 27 May 2024 21:43:16 +0200
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PHP-DEV] [RFC] [Discussion] Add openStream() to
 XML{Reader,Writer}
To: PHP internals <internals@lists.php.net>
References: <CAJp_myWSagrqv3uH6CNxx0qskEsq0Ds-7Srf6DwSvn+4MT8B+w@mail.gmail.com>
 <721C2420-62AB-4586-AFAA-2E831C678EB5@gmail.com>
 <31b39501-e42f-4563-a4c6-1efafdf34512@gmail.com>
Content-Language: en-US
In-Reply-To: <31b39501-e42f-4563-a4c6-1efafdf34512@gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
From: dossche.niels@gmail.com (Niels Dossche)

On 22/05/2024 19:47, Niels Dossche wrote:
> Okay, that seems reasonable to me and I can do that.
> I'll make them return static and do "new static()" like I proposed with the new static method.
> 
> For XMLWriter, I suppose we also want the static counterparts:
> * `XMLWriter::toMemory()` for `(new XMLWriter)->openMemory()`
> * `XMLWriter::toUri()` for `(new XMLWriter)->openUri()`
> 
> If that sounds good I'll update the RFC.
> 
> Niels

I've now updated the RFC to include this kind of change; except I used the term Url instead of Uri.
See the new subsection "Consistency" for what I propose.

Niels