Petr Viktorin <encu...@gmail.com> added the comment:

Hello!
This change added an enum to the stable ABI:
    typedef enum {
        PYGEN_RETURN = 0,
        PYGEN_ERROR = -1,
        PYGEN_NEXT = 1,
    } PySendResult;

Adding new values to enums might break the stable ABI in some (admittedly rare) 
cases; so usually it's better to avoid enums and stick with int and #ifdef'd 
values.
This particular one looks like it won't be extended and won't cause problems, 
but still, switching to int would make stable ABI easier to work with.
Is anyone against switching to int?

See pbo-44727 for details.

----------
nosy: +petr.viktorin

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

Reply via email to