Re: Dérapage image_reduire

Le 28/02/2020 à 08:05, Cerdic a écrit :
> C’est vraiment une histoire d’arrondis car je viens de tester avec un 
> patch de la forme
> ceil($destWidth-0.01)
> ceil($destHeight-0.01)
> 
> et ça corrige aussi toutes les erreurs.
> Maintenant j’hésites à mettre un round.
> Il faudrait retourner dans l’historique de ce morceau de code, mais il 
> faudrait creuser profond j’ai peur :(
> 
> Je pense que l’idée c’est de pas créer une image trop petite quand on va 
> appliquer les rescale derrière : il vaut mieux avoir 1 px de trop que 
> 1px de moins
> Maintenant si je fais une biblio rapide sur internet des pratiques, je 
> trouve de tout : du ceil, du rien du tout (c’est osé), et du round.
> 
> Bref, soit on fix a minima avec le ceil corrigé pour les erreurs 
> d’arrondi, soit on est fou et on prend le risque d’un round.
> 
> Cela dit probablement que l’enjeu est mineur entre les 2 car on manipule 
> des images de plus en plus grandes, et un écart d’1px représente de 
> moins en moins d’erreur relativement à la taille de l’image…
> 
> Si quelqu’un a un avis, qu’il parle maintenant ou se taise à jamais ! :p

Y'aurait pas un lien avec https://core.spip.net/issues/4382 ?

-- 
nicod_