STINNER Victor <vstin...@python.org> added the comment:

sre_constants, sre_compile and sre_parse are not tested and are not documented. 
I don't consider them as public API currently.

If someone has good reason to use them, IMO we must clearly define which exact 
API is needed, properly document and test it.

If we expose something, I don't think that the API would be exposed as 
re.sre_xxx.xxx, but as re.xxx. 

I suggest to hide sre_xxx submodules by adding an underscore to their name. 
Moreover, the "sre_" prefix is now redundant. I suggest renaming:

* sre_constants => re._constants
* sre_compile => re._compile
* sre_parse => re._parse

----------
nosy: +vstinner

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

Reply via email to