Re: r124262 - in _plugins_/scssphp/trunk

en résumé de toute la discussion post PR sur la lib :
is_null($x) (et donc \is_null() si tu as un NameSpace) est totalement optimisé par Zend Compiler
alors que $x === null ne sera optimisé par le Zend Compiler qu’en PHP 8 - mais il est optimisé dans l’opcode, c’est donc pas si mauvais, mais quand même moins bien que is_null($x) en terme de perormance.

Bon faut pas couper les cheveux en 4 non plus, la différence n’a d’intérêt que si tu cherches les derniers pouillèmes et que tu es par exemple dans une boucle avec beaucoup beaucoup d’itérations.

Par contre le is_null() dans un namespace est lui très mauvais, il faut donc faire attention à ce qu’on fait.

On peut dire en conclusion
 * avec $x === null tu sera toujours sur d’être relativement bon et c’est future proof,
 * avec \is_null($x) c’est encore mieux mais lourd à écrire partout
 * avec is_null($x) c’est aussi très bien sans namespace, mais si jamais tu as un namespace ça devient très mauvais...

--
Cédric
Le 20 avr. 2020 à 13:44 +0200, Eric Lupinacci <[email protected]>, a écrit :
> Mais, justement, on ne doit pas privilégier $var === null d’après le PSR ou autre ?
>
> ++
> Eric
>
>
> > Le 20 avr. 2020 à 13:28, Cerdic <[email protected]> a écrit :
> >
> > Hé oui, https://github.com/scssphp/scssphp/pull/98/commits/5b898b75438164d8d3510dd3233ec4b92d3f58d5
> >
> > (c’est la découverte du week end pour moi aussi, mais ça concerne pas trop le code de SPIP car on utilise pas de namespaces)
> >
> > --
> > Cédric
> > Le 20 avr. 2020 à 13:08 +0200, Maïeul Rouquette <[email protected]>, a écrit :
> > > Le 20/04/2020 à 12:49, [email protected] a
> > > écrit :
> > > > Author: Cerdic
> > > > Date: 2020-04-20 10:48:50 +0000 (Mon, 20 Apr 2020)
> > > > New Revision: 124262
> > > >
> > > > Modified:
> > > > _plugins_/scssphp/trunk/
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/Cache.php
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/Colors.php
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/Compiler.php
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/Formatter.php
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/Formatter/Nested.php
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/Node/Number.php
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/Parser.php
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/SourceMap/SourceMapGenerator.php
> > > > _plugins_/scssphp/trunk/lib/scssphp/src/Version.php
> > > > _plugins_/scssphp/trunk/paquet.xml
> > > > Log:
> > > > Mise a jour de la lib ScssPHP avec notamment de belles optimisations de rapidite
> > > >
> > > >
> > > > Details: https://zone.spip.org/trac/spip-zone/changeset/124262
> > > >
> > > C'est normal tout ces antislashs ?
> > >
> > > _______________________________________________
> > > liste: https://listes.rezo.net/mailman/listinfo/spip-dev
> > > doc: https://www.spip.net/
> > > dev: https://core.spip.net/
> > > irc://irc.freenode.net/spip
> > _______________________________________________
> > liste: https://listes.rezo.net/mailman/listinfo/spip-dev
> > doc: https://www.spip.net/
> > dev: https://core.spip.net/
> > irc://irc.freenode.net/spip
>