Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121760
MIME-Version: 1.0
References: <CAH5C8xX5Wzbm1eerFJJ5hCZdUU8rci8NhBwkSwprRKAG531G7Q@mail.gmail.com>
 <CAEKnhAFDYiPoKwjj2J6akEaZ6WgHkaYia9su_fNQGSKidCt=aw@mail.gmail.com>
 <CAFPFaMLWSW+eu8c0v91KLbcZJqcKKDbYZ3OthtNXk805p0AvQA@mail.gmail.com>
 <04DC3709-3B46-41C0-976E-E6AB28BC0D34@edison.tech> <CAFPFaMK+Rm5W3j_W5e3Cjc=HNBcA=U-Va5qbuOXz+q1ccpvswg@mail.gmail.com>
In-Reply-To: <CAFPFaMK+Rm5W3j_W5e3Cjc=HNBcA=U-Va5qbuOXz+q1ccpvswg@mail.gmail.com>
Date: Thu, 23 Nov 2023 01:08:56 +0700
Message-ID: <CANLbj-qQdbbHXyemzQ0FLq-czDXt_bDm6WSptwtxOQ9bzi0k4g@mail.gmail.com>
To: "G. P. B." <george.banyard@gmail.com>
Cc: Mike Schinkel <mike@newclarity.net>, Jakub Zelenka <bukka@php.net>, 
	=?UTF-8?B?TcOhdMOpIEtvY3Npcw==?= <kocsismate90@gmail.com>, 
	PHP Internals List <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [RFC] [Discussion] Resource to object conversion
From: ayesh@php.watch (Ayesh Karunaratne)

> More importantly, it is possible to write cross compatible code, even
> without changing anything about is_resource(), if we convert streams to
> opaque objects.
> It might be tedious and one might need to have redundant instanceof check=
s
> with is_resource() if one does not want to check for a false return, or
> duplicate checks for closed resources.
>


Extensions like GD, IMAP, Curl, and FTP were once very "resource
heavy", but they already received this migration with next to none
disruptions. Shameless plug here
(https://php.watch/articles/resource-object) where I track the
progress. As these changes were merged in the last couple years, I
spent a few hours grepping the Composer top 1000 packages. There were
not that many `is_resource` checks, and it was quite easy and
straightforward to update them to account for the class objects.

I'm supportive of our trajectory so far by slowly but steadily moving
resource objects to class objects. I also support that we do not
modify `is_resource` to mark resource class objects as a resource. I
really look forward to the day that we remove `is_resource` and kill
the final resource object in PHP =F0=9F=A5=BA.