Produits Entr'ouvert » Authentic 2

Met l'affaire "FileType" dans les tests directement, inutile de "pourrir" authentic.compat avec ça.

Thomas Noël a écrit :

Quand même, je ne pige pas trop le remplacement des simple « x=file.read » par des tordus « with open as f: x=f.read »

C'est très simple, les chances de voir le patch acké par Benj sont multipliées par 3 si on parvient à placer des gestionnaires de contexte ☺
(ça permet aussi d'éviter une fermeture explicite du fichier, cette fermeture étant effectuée à la sortie du contexte.)

Paul Marillonnet a écrit :

(ça permet aussi d'éviter une fermeture explicite du fichier, cette fermeture étant effectuée à la sortie du contexte.)

Sauf erreur de ma part, open().read() ferme le fichier aussitôt après le read. Non ? Ah, je découvre que c'est de la magie CPython.

Ok pour le with, donc.

Ok oui, je pensais à un équivalent sans gestionnaire de contexte :

f = open(...)
f.read()

Thomas Noël a écrit :

Sauf erreur de ma part, open().read() ferme le fichier aussitôt après le read. Non ? Ah, je découvre que c'est de la magie CPython.

La réponse c'est peut-être¹; mais avec with c'est certain et c'est comme ça qu'il faut faire et ça permet de prendre de bonnes habitudes de gestion des ressources en général, et en fait je trouve ça joli à force. Logiquement le compteur de référence doit tomber à zéro en sortie de scope et le descripteur de fichier être fermé, mais si il y a une exception dans read() ça peut ne pas le faire pour diverses raisons.

¹ https://stackoverflow.com/questions/4599980/python-close-file-descriptor-question

ducktype powa ! remplacé isinstance(x, file) par hasattr(x, 'read').