Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:123690
To: "internals@lists.php.net" <internals@lists.php.net>
Subject: [PHP-DEV] FFI::new(), FFI::type(), FFI::cast() statically RFC is stupid implementation
Thread-Topic: FFI::new(), FFI::type(), FFI::cast() statically RFC is stupid
 implementation
Thread-Index: AQHawr3AQUllAdiBrEi5G5lYKsqtsA==
X-MS-Exchange-MessageSentRepresentingType: 1
Date: Thu, 20 Jun 2024 03:01:58 +0000
Message-ID:
	<TYWP286MB2251D09C79295CA010564C42F2C82@TYWP286MB2251.JPNP286.PROD.OUTLOOK.COM>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-Exchange-Organization-SCL: -1
X-MS-TNEF-Correlator:
X-MS-Exchange-Organization-RecordReviewCfmType: 0
msip_labels:
Content-Type: multipart/alternative;
	boundary="_000_TYWP286MB2251D09C79295CA010564C42F2C82TYWP286MB2251JPNP_"
Precedence: bulk
list-help: <mailto:internals+help@lists.php.net
list-unsubscribe: <mailto:internals+unsubscribe@lists.php.net>
list-post: <mailto:internals@lists.php.net>
List-Id: internals.lists.php.net
MIME-Version: 1.0
From: chopins.xiao@gmail.com (chopins xiao)

--_000_TYWP286MB2251D09C79295CA010564C42F2C82TYWP286MB2251JPNP_
Content-Type: text/plain; charset="gb2312"
Content-Transfer-Encoding: base64

SGF2ZW4ndCBiZWVuIHBheWluZyBhdHRlbnRpb24gZm9yIGEgd2hpbGUgYW5kIGZvdW5kIHRoYXQg
RkZJOjpuZXcoKSwgRkZJOjp0eXBlKCksIEZGSTo6Y2FzdCgpc3RhdGljYWxseSBhcmUgbWFya2Vk
IGFzIGRlcHJlY2F0ZWQsIHdoaWNoIGlzIGEgYmFkIGltcGxlbWVudGF0aW9uLg0KU3VnZ2VzdGlu
ZyBpbiBSRkM6aHR0cHM6Ly93aWtpLnBocC5uZXQvcmZjL2RlcHJlY2F0ZV9mdW5jdGlvbnNfd2l0
aF9vdmVybG9hZGVkX3NpZ25hdHVyZXMjZmZpY2FzdF9mZmluZXdfYW5kX2ZmaXR5cGUgdGhhdCB5
b3Ugc2hvdWxkIHJlcGxhY2Ugc3RhdGljIGNhbGxzIHdpdGggaW5zdGFuY2VzIGlzIGEgc3R1cGlk
IHN1Z2dlc3Rpb24uIFRoaXMgUkZDIHdhcyBpbXBsZW1lbnRlZCBvbiBKdWx5IDE4LCAyMDIzIGlu
IGh0dHBzOi8vZ2l0aHViLmNvbS9waHAvcGhwLXNyYy9jb21taXQvNGFjZjAwODRkY2Q2M2VjMzY5
YTYxMGVjOTY2ZGIzM2YzMjI2OTRjOC4gT24gdGhpcyBpc3N1ZSwgSSBzdWJtaXR0ZWQgYSBwcm9w
b3NhbCBvbiBNYXkgMTQsIDIwMjIgaHR0cHM6Ly9naXRodWIuY29tL3BocC9waHAtc3JjL2lzc3Vl
cy84NTU0IGhlcmUuDQoNCkhlcmUncyB3aHkgaXQncyBub3QgYWR2aXNhYmxlIHRvIHJlbW92ZSBz
dGF0aWM6DQoNCiAgMS4gIFRoZSBub24tc3RhdGljIGNhbGwgdGhyb3VnaCB0aGUgRkZJIGluc3Rh
bmNlIGlzIHRoZSBDIGZ1bmN0aW9uLCBhbmQgdGhlIG5ldygpLCB0eXBlKCksIGNhc3QoKSBhcmUg
UEhQIGNsYXNzIG1ldGhvZHMsIHdoaWNoIGlzIGEgdmVyeSBzdHJhbmdlIEFQSSBpbXBsZW1lbnRh
dGlvbiwgYW5kIHRoZXNlIHRocmVlIGZ1bmN0aW9uIG5hbWVzIG9jY3VweSB0aGUgQyBmdW5jdGlv
biB0aGF0IG1heSBleGlzdC4gQWxzbywgdGhlIGZ1bmN0aW9uIHplbmRfZmZpX2dldF9mdW5jIGlt
cGxlbWVudGF0aW9uIGluIGZmaS5jIGlzIG5vdCBzbyBnb29kIG9uIGl0cyBvd24uDQogIDIuICBX
aGVuIHlvdSBuZWVkIHRvIGdldCBub24tUEhQIHR5cGUgZGF0YSwgeW91IGNhbiBnZXQgaXQgZGly
ZWN0bHkgdGhyb3VnaCBzdGF0aWMgbmV3IHdpdGhvdXQgZ2V0dGluZyBhbiBGRkkgb2JqZWN0IGlu
c3RhbmNlIGZpcnN0LiBUaGlzIGlzIHVzZWZ1bCB3aGVuIHVzaW5nIG9ubHkgQyBwcmltaXRpdmUg
ZGF0YSB0eXBlcywgd2l0aG91dCB0aGlua2luZyBhYm91dCBwZXJmb3JtYW5jZSwgYW5kIHdpdGhv
dXQgY2FsbGluZyBDIGxpYnJhcmllcy4NCiAgMy4gIFBIUCBwcm9ncmFtbWVycyBhcmVuJ3QgYWxs
IGZvb2xzLCBQSFAgcHJvZ3JhbW1lcnMgd2hvIGNhbiB1c2UgdGhlIEZGSSBpbnRlcmZhY2UgbXVz
dCBoYXZlIHRoZSByZWxldmFudCBza2lsbHMuDQoNCg==

--_000_TYWP286MB2251D09C79295CA010564C42F2C82TYWP286MB2251JPNP_
Content-Type: text/html; charset="gb2312"
Content-Transfer-Encoding: quoted-printable

<html>
<head>
<meta http-equiv=3D"Content-Type" content=3D"text/html; charset=3Dgb2312">
<style type=3D"text/css" style=3D"display:none;"> P {margin-top:0;margin-bo=
ttom:0;} </style>
</head>
<body dir=3D"ltr">
<div style=3D"margin-top: 1em; margin-bottom: 1em;">
<div style=3D"direction: ltr; font-family: Calibri, Helvetica, sans-serif; =
font-size: 12pt; color: rgb(0, 0, 0);">
Haven't been paying attention for a while and found that <code>FFI::new()</=
code>,
<code>FFI::type()</code>, <code>FFI::cast()</code>statically&nbsp;are marke=
d as deprecated, which is a bad implementation.<br>
Suggesting in RFC:<a href=3D"https://wiki.php.net/rfc/deprecate_functions_w=
ith_overloaded_signatures#fficast_ffinew_and_ffitype" id=3D"OWA815a4cca-667=
7-1623-cd90-749d57bc9ce6" class=3D"OWAAutoLink" rel=3D"nofollow">https://wi=
ki.php.net/rfc/deprecate_functions_with_overloaded_signatures#fficast_ffine=
w_and_ffitype</a>&nbsp;that
 you should replace static calls with instances is a stupid suggestion. Thi=
s RFC was implemented on July 18, 2023 in
<a href=3D"https://github.com/php/php-src/commit/4acf0084dcd63ec369a610ec96=
6db33f322694c8" id=3D"LPlnk">
https://github.com/php/php-src/commit/4acf0084dcd63ec369a610ec966db33f32269=
4c8</a>. On this issue, I submitted a proposal on May 14, 2022
<a href=3D"https://github.com/php/php-src/issues/8554" id=3D"LPlnk235224">h=
ttps://github.com/php/php-src/issues/8554</a>&nbsp;here.</div>
<div style=3D"direction: ltr; font-family: Calibri, Helvetica, sans-serif; =
font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style=3D"direction: ltr; font-family: Calibri, Helvetica, sans-serif; =
font-size: 12pt; color: rgb(0, 0, 0);">
Here's why it's not advisable to remove static:</div>
</div>
<ol start=3D"1" style=3D"direction: ltr;">
<li style=3D"font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; =
color: rgb(0, 0, 0);">
<span style=3D"font-family: Calibri, Helvetica, sans-serif; font-size: 12pt=
; color: rgb(0, 0, 0);">The non-static call through the FFI instance is the=
 C function, and the
<code>new()</code>, <code>type()</code>, <code>cast()</code> are PHP class =
methods, which is a very strange API implementation, and these three functi=
on names occupy the C function that may exist. Also, the function
<code>zend_ffi_get_func</code>&nbsp;implementation in <code>ffi.c</code>&nb=
sp;is not so good on its own.</span></li><li style=3D"font-family: Calibri,=
 Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<span style=3D"font-family: Calibri, Helvetica, sans-serif; font-size: 12pt=
; color: rgb(0, 0, 0);">When you need to get non-PHP type data, you can get=
 it directly through static
<code>new</code> without getting an FFI object instance first. This is usef=
ul when using only C primitive data types, without thinking about performan=
ce, and without calling C libraries.</span></li><li style=3D"font-family: C=
alibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<span style=3D"font-family: Calibri, Helvetica, sans-serif; font-size: 12pt=
; color: rgb(0, 0, 0);">PHP programmers aren't all fools, PHP programmers w=
ho can use the FFI interface must have the relevant skills.</span></li></ol=
>
<div style=3D"direction: ltr; margin-top: 1em; margin-bottom: 1em; font-fam=
ily: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"=
 class=3D"elementToProof">
<br>
</div>
<div id=3D"Signature"></div>
</body>
</html>

--_000_TYWP286MB2251D09C79295CA010564C42F2C82TYWP286MB2251JPNP_--