Amaury Forgeot d'Arc <amauryfa <at> gmail.com> writes:

> I'd prefer it was written :
>        if (PyUnicode_READY(*filename) < 0)
> because "< 0" clearly indicates an error condition.

Why not just have it return 0 on error? This would be more consistent with API
functions that return "false" values like NULL and would just be

if (!PyUnicode_READY(s)) return NULL;

in code.

Regards,
Benjamin




_______________________________________________
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com

Reply via email to