René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Unify `PyUncode_Count` and `unicode_count`","articleBody":"# Feature or enhancement\r\n\r\nRight now `PyUnicode_Count` from https://github.com/python/cpython/blob/cbdeda8ce7a3543cb3376d70e4cd46fcf24f42a7/Objects/unicodeobject.c#L8968-L9040 and `unicode_count` from https://github.com/python/cpython/blob/cbdeda8ce7a3543cb3376d70e4cd46fcf24f42a7/Objects/unicodeobject.c#L10854-L10916 share a lot of code.\r\n\r\nThey can be unified, because the do the same thing.\r\n\r\n# Pitch\r\n\r\n[Citing](https://github.com/python/cpython/pull/96929#issuecomment-1270262538) @encukou:\r\n\r\n\u003e Apparently unicode_count missed [an optimization in 2011](https://github.com/python/cpython/commit/c3cec7868bf1019c0987f1e9aadb56d73fa93d61), otherwise they're equivalent (except arg parsing \u0026 converting the return value). Merging them could add the optimization to unicode_count.\r\nIf you want to work on that, note that there's also anylib_count that duplicates the main switch.\r\n\r\n# Previous discussion\r\n\r\n\u003c!--\r\n  New features to Python should first be discussed elsewhere before creating issues on GitHub,\r\n  for example in the \"ideas\" category (https://discuss.python.org/c/ideas/6) of discuss.python.org,\r\n  or the python-ideas mailing list (https://mail.python.org/mailman3/lists/python-ideas.python.org/).\r\n  Use this space to post links to the places where you have already discussed this feature proposal:\r\n--\u003e\r\n\r\nLink: https://github.com/python/cpython/pull/96929\r\n\r\n\u003c!--\r\nYou can freely edit this text. Remove any lines you believe are unnecessary.\r\n--\u003e\r\n\r\nPR in the works.","author":{"url":"https://github.com/sobolevn","@type":"Person","name":"sobolevn"},"datePublished":"2022-10-06T18:06:05.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/97982/cpython/issues/97982"}