Berker Peksag added the comment:

I think the difference between xxlimited and xxmodule is still a bit ambiguous. 
I would expect a mention of PEP 384 or a link to 
https://docs.python.org/3/c-api/stable.html to explain what Py_LIMITED_API 
really is.

+#ifndef Py_LIMITED_API
+# error "This file requires Py_LIMITED_API"
+#endif

I don't have a strong opinion about this, but I think the error message could 
be more informative (e.g. you must define a Py_LIMITED_API macro) Or we can 
improve https://docs.python.org/3/c-api/stable.html to give an example there.

----------
nosy: +berker.peksag, serhiy.storchaka
stage:  -> patch review
type:  -> enhancement

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue16731>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to