On Mon, Jun 25, 2018 at 9:32 PM, Sara Golemon <poll...@php.net> wrote:
> On Sun, Jun 24, 2018 at 11:47 AM, Nikita Popov <nikita....@gmail.com> > wrote: > > https://wiki.php.net/rfc/deprecations_php_7_3 > > > > Undocumented mbstring function aliases > > > Yeah, if they're just dumb aliases, then it's a slight gain to narrow > the symbol table by removing duplicates. A modest composer package > (nay, include file) would be sufficient to provide a bridge for > existing projects which use the removed aliases once they're gone. > This deprecation isn't *needed*, but it's also fairly low impact. 🤷 > > > String search functions with integer needle > > > Agree with stas, the wtf quotient on this one is high. Mitigating > this is the availability of strict types, but even still I think > deprecating the behavior altogether is a wise move. Callers can > easily insert a call to chr() (or IntlChar::chr() ) to maintain the > old behavior in a version agnostic way. 😕 > > > fgetss() function and string.strip_tags filter > > > "...these functions seem to be of very little utility..." [citation needed] > Anecdotally I struggle to imagine the use-case, but my imagination > isn't what it once was. There's not a trivial workaround for someone > who's dealing with a large/streaming data source from which they want > to strip tags. Deprecating this should come at a higher standard of: > proving that it is used, providing a userspace replacement, or both. > 😤 > As an insight into how much this functionality is used from a somewhat unexpected direction: While implementing the deprecation, I found out that the string.strip_tags filter does not have a single test and has been segfaulting on construction since PHP 7.0 if no allowed tags are specified (and performed OOB reads if they were specified). These issues have been fixed in https://github.com/php/php-src/commit/791f07e4f06a943bd7892bdc539a7313fb3d6d1e . Nikita