RetroSearch Browse

Thu Oct 6 14:57:03 CEST 2011 · https://mail.python.org/pipermail/python-dev/2011-October/113878.html

Hi,

with the new Unicode API, there are many checks like:
+        if (PyUnicode_READY(*filename))
+            goto handle_error;

Every time I read it, I get it wrong:
   "If filename is ready, then fail"
then I have to remember that the function returns either 0 or -1.

I'd prefer it was written :
       if (PyUnicode_READY(*filename) < 0)
because "< 0" clearly indicates an error condition.
That's how all calls to PyType_Ready are written, for example.

Am I the only one to be distracted by this idiom?

-- 
Amaury Forgeot d'Arc

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from https://mail.python.org/pipermail/python-dev/2011-October/113878.html below:

[Python-Dev] check for PyUnicode_READY look backwards